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

meta: add Prettier #4707

Merged
merged 8 commits into from
Sep 28, 2023
Merged

meta: add Prettier #4707

merged 8 commits into from
Sep 28, 2023

Conversation

aduh95
Copy link
Contributor

@aduh95 aduh95 commented Sep 27, 2023

JS files are ignored for now because we want to migrate them to TS anyway.
MD files are also ignored to minimize the diff.

Nothing is enforcing use of Prettier for now (so it doesn't reject PRs), maybe it should?

JS files are ignored for now because we want to migrate them to TS
anyway.
@socket-security
Copy link

socket-security bot commented Sep 27, 2023

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
eslint-config-prettier 9.0.0 environment +0 20.8 kB lydell
prettier 2.7.1 None +0 15.8 MB prettier-bot

Copy link
Member

@Murderlon Murderlon left a comment

Choose a reason for hiding this comment

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

I like the idea 👌

.eslintrc.js Outdated Show resolved Hide resolved
@aduh95 aduh95 mentioned this pull request Sep 28, 2023
package.json Outdated Show resolved Hide resolved
@@ -18,6 +18,6 @@ <h1>Test apps</h1>
<li><a href="dashboard-ui/index.html">dashboard-ui</a></li>
<li><a href="dashboard-vue/index.html">dashboard-vue</a></li>
</ul>
<nav>
</nav>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Prettier already caught an HTML syntax error

@aduh95 aduh95 merged commit 2f691ac into main Sep 28, 2023
18 checks passed
@aduh95 aduh95 deleted the prettier branch September 28, 2023 14:47
@arturi
Copy link
Contributor

arturi commented Sep 28, 2023

Approved!

@github-actions github-actions bot mentioned this pull request Sep 29, 2023
github-actions bot added a commit that referenced this pull request Sep 29, 2023
| Package                   | Version | Package                   | Version |
| ------------------------- | ------- | ------------------------- | ------- |
| @uppy/audio               |   1.1.3 | @uppy/store-default       |   3.0.4 |
| @uppy/aws-s3              |   3.3.1 | @uppy/store-redux         |   3.0.4 |
| @uppy/aws-s3-multipart    |   3.7.0 | @uppy/svelte              |   3.1.0 |
| @uppy/companion           |   4.9.1 | @uppy/thumbnail-generator |   3.0.5 |
| @uppy/companion-client    |   3.4.1 | @uppy/transloadit         |   3.3.1 |
| @uppy/compressor          |   1.0.4 | @uppy/tus                 |   3.3.1 |
| @uppy/core                |   3.5.1 | @uppy/utils               |   5.5.1 |
| @uppy/dashboard           |   3.5.4 | @uppy/webcam              |   3.3.3 |
| @uppy/image-editor        |   2.2.1 | @uppy/xhr-upload          |   3.4.1 |
| @uppy/remote-sources      |   1.0.4 | uppy                      |  3.17.0 |

- meta: add Prettier (Antoine du Hamel / #4707)
- @uppy/aws-s3-multipart: retry signature request (Merlijn Vos / #4691)
- meta: update linter config to cover more files (Mikael Finstad / #4706)
- @uppy/image-editor: ImageEditor.jsx - remove 1px black lines (Evgenia Karunus / #4678)
- meta: delete `.yarn/releases/yarn-3.4.1.cjs` (Antoine du Hamel)
- meta: fix linter errors (Antoine du Hamel / #4704)
- @uppy/utils: test: migrate to Vitest for Uppy core and Uppy plugins (Antoine du Hamel / #4700)
- meta: run corepack yarn (Mikael Finstad)
- @uppy/companion: upgrade TS target (Mikael Finstad / #4670)
- @uppy/companion: use deferred length for tus streams (Mikael Finstad / #4697)
- @uppy/companion-client: fix a refresh token race condition (Mikael Finstad / #4695)
- meta: add companion hotfix doc (Mikael Finstad / #4683)
- meta: run type checks also for companion and add files to docker (Mikael Finstad / #4688)
- @uppy/svelte: revert breaking change (Antoine du Hamel / #4694)
- meta: Update yarn.lock (Artur Paikin)
- @uppy/companion: fix instagram/facebook auth error regression (Mikael Finstad / #4692)
- @uppy/aws-s3-multipart: aws-s3-multipart - call `#setCompanionHeaders` in `setOptions` (jur-ng / #4687)
- @uppy/svelte: Upgrade Svelte to 4 (frederikhors / #4652)
- @uppy/companion: add test endpoint for dynamic oauth creds (Mikael Finstad / #4667)
- meta: fix VITE_COMPANION_ALLOWED_HOSTS (Mikael Finstad / #4690)
- @uppy/companion: fix edge case for pagination on root (Mikael Finstad / #4689)
- @uppy/companion: fix onedrive pagination (Mikael Finstad / #4686)
Murderlon added a commit that referenced this pull request Oct 2, 2023
* main:
  Bucket fn also remote files (#4693)
  fixup! Added Companion OAuth Key type (#4668)
  Added Companion OAuth Key type (#4668)
  meta: check for formatting in CI (#4714)
  meta: bump get-func-name from 2.0.0 to 2.0.2 (#4709)
  meta: run Prettier on existing files (#4713)
  Release: uppy@3.17.0 (#4716)
  meta: add Prettier (#4707)
  @uppy/aws-s3-multipart: retry signature request (#4691)
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.

3 participants