Skip to content

♻️ Remove unused imports#35435

Merged
samouri merged 2 commits intoampproject:mainfrom
benmccann:patch-1
Aug 3, 2021
Merged

♻️ Remove unused imports#35435
samouri merged 2 commits intoampproject:mainfrom
benmccann:patch-1

Conversation

@benmccann
Copy link
Copy Markdown
Contributor

@benmccann benmccann commented Jul 28, 2021

When building my project with Rollup, I get the warning:

default imported from external module 'url' but never used
default imported from external module 'querystring' but never used

@amp-owners-bot amp-owners-bot bot requested a review from banaag July 28, 2021 15:42
@amp-owners-bot
Copy link
Copy Markdown

amp-owners-bot bot commented Jul 28, 2021

Hey @ampproject/wg-caching! These files were changed:

validator/js/nodejs/index.js

@benmccann benmccann changed the title ♻️ Remove unused import ♻️ Remove unused imports Jul 28, 2021
@Gregable Gregable requested review from Gregable and removed request for banaag July 28, 2021 15:55
@samouri
Copy link
Copy Markdown
Member

samouri commented Jul 28, 2021

@benmccann: FYI there have been CI changes in main since you created the PR, so I've rebased and force pushed

@samouri samouri enabled auto-merge (squash) July 28, 2021 18:34
@benmccann
Copy link
Copy Markdown
Contributor Author

Thanks @samouri. It looks like there's some bundle size job that's hanging. Can you restart that or merge manually?

@samouri
Copy link
Copy Markdown
Member

samouri commented Aug 3, 2021

Thanks @samouri. It looks like there's some bundle size job that's hanging. Can you restart that or merge manually?

I'm on it! The fix for the CI bug was merged yesterday. Thanks for staying on top of it, I'll make sure this is merged today

@samouri samouri merged commit 8d0c2f7 into ampproject:main Aug 3, 2021
westonruter added a commit to westonruter/amphtml that referenced this pull request Aug 3, 2021
…tok-validation

* 'main' of github.com:ampproject/amphtml: (72 commits)
  build: run amp lint --fix to address import order of jixie (ampproject#35513)
  ✨ [amp-analytics] Add Custom Browser Event Tracker (ampproject#35193)
  babel: teach amp mode transformer about #core/mode (ampproject#35477)
  🚮 Remove experiment `amp-consent-granular-consent` (ampproject#35508)
  ♻️ Enable auto-sorting+grouping within src/ and 3p/ (ampproject#35454)
  🐛  [amp-render] fix root-element stripping from amp-render with amp-bind (ampproject#35449)
  ✅ [Story interactive] Add Example Story for Detailed Results Component (ampproject#35450)
  🐛 Fix error on bento example (ampproject#35490)
  🐛 amp-story-grid-layer: Fix AMP invalidation error in documentation (ampproject#35503)
  🐛 Fix code formatting (ampproject#35499)
  ✅ buildDom: add tests for amp-fit-text and amp-layout (ampproject#35494)
  ♻️ Remove unused imports (ampproject#35435)
  ✨ amp-connatix-player: iframe domain based on a property (ampproject#35179)
  Updated document with use cases of remote config (ampproject#35496)
  AMP.goBack: update documentation (ampproject#29290)
  🏗 Allow the bundle-size job to run even if the builds were skipped (ampproject#35492)
  build-system: improve terser/esbuild integration (ampproject#35466)
  🧪 [Story performance] Disable animations on first page to 50% (ampproject#35476)
  📖 [Amp story] [Page attachments] Amp.dev Docs for New Page Attachment Features ampproject#34883 (ampproject#35338)
  🚀 [Story interactive] Rewrite Image URL to Cached URL for Image Quizzes and Polls (ampproject#35375)
  ...
@benmccann benmccann deleted the patch-1 branch August 3, 2021 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants