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

chore(galaxy): use Node script instead of bash script #2118

Merged
merged 4 commits into from
Jun 17, 2024

Conversation

hanspagel
Copy link
Member

Currently, we use a bash script and the CLI to build/format the @scalar/galaxy package. If this breaks, for some reason, it’s hard to fix and blocks CI.

This PR uses a Node script instead. If that breaks and blocks CI, it’s easy to modify.

Note: This is extracted from #2091.

@hanspagel hanspagel requested a review from marclave as a code owner June 17, 2024 09:06
Copy link

changeset-bot bot commented Jun 17, 2024

🦋 Changeset detected

Latest commit: e49cca7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@scalar/galaxy Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

relativeci bot commented Jun 17, 2024

#2181 Bundle Size — 3.96MiB (0%).

e49cca7(current) vs da0b72b main#2176(baseline)

Bundle metrics  no changes
                 Current
#2181
     Baseline
#2176
No change  Initial JS 3.96MiB 3.96MiB
No change  Initial CSS 0B 0B
Change  Cache Invalidation 0% 100%
No change  Chunks 1 1
No change  Assets 1 1
No change  Modules 1231 1231
No change  Duplicate Modules 0 0
No change  Duplicate Code 0% 0%
No change  Packages 155 155
No change  Duplicate Packages 0 0
Bundle size by type  no changes
                 Current
#2181
     Baseline
#2176
No change  JS 3.96MiB 3.96MiB

Bundle analysis reportBranch chore/use-node-script-instead-of...Project dashboard

Copy link
Member

@marclave marclave left a comment

Choose a reason for hiding this comment

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

nice :) tested and works!

@marclave marclave merged commit c185f10 into main Jun 17, 2024
9 checks passed
@marclave marclave deleted the chore/use-node-script-instead-of-bash-script branch June 17, 2024 19:01
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

3 participants