Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update message to remove duplicate is #2466

Merged
merged 1 commit into from
Apr 23, 2020

Conversation

AdarshKonchady
Copy link
Contributor

@AdarshKonchady AdarshKonchady commented Apr 23, 2020

image

I was getting this message on a workbox build. Notice the duplicate 'is'.

R: @jeffposnick @philipwalton

@jeffposnick jeffposnick self-requested a review April 23, 2020 13:56
Copy link
Contributor

@jeffposnick jeffposnick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@jeffposnick jeffposnick merged commit a8c6639 into GoogleChrome:master Apr 23, 2020
jeffposnick added a commit that referenced this pull request Jul 8, 2020
* Update message to remove duplicate `is` (#2466)

* v5.1.3

* gulp-glitch first pass (#2433)

* removed old demos
moved new demos on glitch into src folder

* Removed gulp tasks

* Linting

* updated to 5.0.0

* clean commit of publish-glitch w/o
problematic secrets in history

* Added express server to top level
Requires explanation w/in server

* updated background sync
strategies
precaching
w/ new updateServer.js

* incrementing versions

* updated precache express server

* error handling in publish-glitch.js
fixed error in workbox-core
converted:
   workbox-cacheable-response
   workbox-expiration
   workbox-routing

* tested pushing new version

* updated:
  workbox-google-analytics
  workbox-range-requqests
  workbox-streams
modified publish-glitch logging

* updated final demos:
   workbox-broadcast-update
   workbox-navigation-preload
   workbox-window

Co-authored-by: Jeff Posnick <jeffy@google.com>

* resetting to upstream/master

* pushing initial conversion attempt for workbox-cli

* made changes per Jeff's comments

* Removed some accidental files

* Add a TS build step

* WIP

* Cleaned up tests

* Linting

* More linting

* package.json test fix

Co-authored-by: Adarsh Konchady <akonchady@gmail.com>
Co-authored-by: Jeff Posnick <jeffy@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants