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.

Sign up for GitHub

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

Jump to bottom

feat: Add script to check if packages pass publint #523

Merged
merged 59 commits into from
Dec 18, 2023
Merged

feat: Add script to check if packages pass publint #523

merged 59 commits into from
Dec 18, 2023

Conversation

lachlancollins
Copy link
Collaborator

🎯 Changes

https://publint.dev/ lints npm packages to ensure widest compatibility across environments. This data would help surface issues and prompt contributions to correct these issues.

✅ Checklist

  • I have given my PR a descriptive title
  • I have run pnpm run lint locally on my changes

lachlancollins and others added 30 commits November 6, 2023 09:08
@lachlancollins
Add zod schemas for CI validation
9fff772
@lachlancollins
Require npm field for components.json
14f7624
@lachlancollins
Remove svelte-layout-resizable
1e2edc5
@lachlancollins
Stricter Zod validation
f97d23f
@lachlancollins
Stricter regex
9c5242a 
https://www.npmjs.com/package/package-name-regex
@lachlancollins
Stricter repository field validation
5d4c9f3
@lachlancollins
Merge branch 'main' into require-npm-field
c668c8f
@lachlancollins
Implement requested changes
54cee2a
@lachlancollins
Add back accidentally removed field
8386ee2
@lachlancollins
Move SvelteStore to templates.json
6cf2e8b
@lachlancollins
Update category and tags
b593067
@lachlancollins
Merge branch 'main' into require-npm-field
77d7f68
@lachlancollins
Merge branch 'main' into require-npm-field
0c9a44b
@lachlancollins
Add script to get npm data
ac4f44a
@lachlancollins
Add script to get publint data
cc04fa8
@lachlancollins
Merge branch 'main' into update-npm-data
30880ac
@lachlancollins
Merge branch 'update-npm-data' into update-publint-data
cc2a5a2
@lachlancollins
Merge branch 'main' into update-npm-data
96116de
@lachlancollins
Re-run updateNpm.js
20591ca
@lachlancollins
Merge branch 'update-npm-data' into update-publint-data
e30e043
@lachlancollins
Re-run updatePublint.js
31b32ca
@lachlancollins
Merge branch 'main' into update-npm-data
9c8a894
@lachlancollins
Implement initial feedback
c349b57
@lachlancollins
Merge branch 'update-npm-data' into update-publint-data
0dd3d72
@lachlancollins
Use npm.json data
b4e4585
@lachlancollins
Update npm.js
64ab391
@lachlancollins
Merge branch 'update-npm-data' into update-publint-data
42f7820
@lachlancollins @MacFJA
Switch async/await to then/catch
d6589a4 
Co-authored-by: MacFJA <[email protected]>
@lachlancollins
Fix prettier
50630f3
@lachlancollins
Run script
0c00771
lachlancollins and others added 22 commits December 17, 2023 16:08
@lachlancollins
Re-run updateNpm
2e46d12
@lachlancollins
Merge branch 'update-npm-data' into update-publint-data
afd357a
@lachlancollins
Also read tools.json
77a078a
@lachlancollins
Add @types/node
0aff9bb
@lachlancollins
Restructure npm.json output
2379a37
@lachlancollins
Merge branch 'update-npm-data' into update-publint-data
77a7828
@lachlancollins
Fix updatePublint.js
04d8a9b
@lachlancollins
Display last update on components page
2f7d1bb
@lachlancollins
Add to weekly workflow
6934616
@lachlancollins
Clarify updating npm data
8972dd6
@lachlancollins
Merge branch 'main' into update-npm-data
9094497
@lachlancollins
Merge branch 'update-npm-data' into update-publint-data
1c8648c
@lachlancollins @MacFJA
Update src/lib/utils/injectNpmData.ts
0f4ea68 
Co-authored-by: MacFJA <[email protected]>
@lachlancollins
Smaller date font, add version
2813b31
@lachlancollins
Improve error text
0d6202f
@lachlancollins
Fix property title
37131b4
@lachlancollins
Merge branch 'main' into update-npm-data
4853733
@lachlancollins
Merge branch 'update-npm-data' into update-publint-data
076e6e3
@lachlancollins
Merge branch 'main' into update-publint-data
3cb639b
@lachlancollins
Move json to lib/data directory
148f5f9
@lachlancollins
Fix npm.json path
e26e761
@lachlancollins
Also check tools.json
ee955ad
@netlify Netlify
Copy link

netlify bot commented Dec 18, 2023 •
edited
Loading

✅ Deploy Preview for sveltesociety-preview ready!

Name Link
🔨 Latest commit cb90957
🔍 Latest deploy log https://app.netlify.com/sites/sveltesociety-preview/deploys/657fa06891f0860008cbf296
😎 Deploy Preview https://deploy-preview-523--sveltesociety-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@lachlancollins lachlancollins changed the title feat: Add script to check if packages pass publint tests feat: Add script to check if packages pass publint Dec 18, 2023
@lachlancollins lachlancollins merged commit 156f654 into svelte-society:main Dec 18, 2023
5 checks passed
@lachlancollins lachlancollins deleted the update-publint-data branch December 18, 2023 02:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@lachlancollins