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/dedicated-trigger-of-downstream-jobs #111

Merged
merged 5 commits into from
Mar 26, 2024

Conversation

@collincchoy
Copy link
Contributor

hey FYI @jimid27 , this repo/package is a bit weird compared to the others. For example, there is no build script in here so npm run build fails. it's basically just a config file that's version controlled.

Interestingly in terms of impact, this is probably at the tippy top of the our dependency chain tho so it actually is the most friction to bump everywhere and it isn't bumped often.

@jimid27
Copy link
Contributor Author

jimid27 commented Mar 22, 2024

hey FYI @jimid27 , this repo/package is a bit weird compared to the others. For example, there is no build script in here so npm run build fails. it's basically just a config file that's version controlled.

Interestingly in terms of impact, this is probably at the tippy top of the our dependency chain tho so it actually is the most friction to bump everywhere and it isn't bumped often.

Ah ok good intel! What does the "preferred" release strategy look like for this repo, if any?

@collincchoy
Copy link
Contributor

collincchoy commented Mar 22, 2024

I think the workflow used to just be a straight publish. There are definitely other upstream npm package dependencies here too so i'd guess like, setup node -> npm ci -> publish is fine.

i see that you're breaking out the upstream version bumps from the shared action which I imagine should make having a little custom gha workflow here easier? so we could just fallback to a previous version of the release workflow that's just a "basic checkout -> setup node -> install deps -> publish package" -> fan out trigger downstream version bumping jobs?

we could instead try to tweak the shared action to effectively skip the build step - i don't feel strongly on which direction to take but I'd imagine it just adds a weird, maybe-unnecessary layer of complexity into that action to be like "skip_build=true" 🤷 .

@jimid27
Copy link
Contributor Author

jimid27 commented Mar 22, 2024

I think the workflow used to just be a straight publish. There are definitely other upstream npm package dependencies here too so i'd guess like, setup node -> npm ci -> publish is fine.

i see that you're breaking out the upstream version bumps from the shared action which I imagine should make having a little custom gha workflow here easier? so we could just fallback to a previous version of the release workflow that's just a "basic checkout -> setup node -> install deps -> publish package" -> fan out trigger downstream version bumping jobs?

we could instead try to tweak the shared action to effectively skip the build step - i don't feel strongly on which direction to take but I'd imagine it just adds a weird, maybe-unnecessary layer of complexity into that action to be like "skip_build=true" 🤷 .

I think just adding a skip build input that defaults to false is the cleanest solution here! And we can just set it to true for eslint-config

Comment on lines 35 to 41
- uses: prefecthq/actions-trigger-downstream-npm-package-updates@main
id: trigger-downstream-npm-package-update-prefect-oss
if: ${{ steps.release-ui-components.publish.outputs.type }}
with:
GITHUB_TOKEN: ${{ secrets.PREFECT_ACTIONS_RW }}
DOWNSTREAM_REPO_NAME: prefect
RELEASE_TAG: ${{ steps.release-ui-components.publish.outputs.version}}
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this action at a point that we could plop in all of the other downstream packages (pretty much all of our npm packages use this) so

  • prefect-design
  • prefect-ui-library
  • graphs
  • vue-compositions

or do we also need separate secrets for each still?

(😅 - i'd like to cut a new version of this "soon". suuuuuper no rush tho. "soon" could be really whenever, but it'd be neat to test all of this out with the next version bump)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We would need to create GH secrets still but I can get that done today/monday!

Copy link
Contributor

Choose a reason for hiding this comment

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

that would be dope!

will save me so many button clicks!!!

@jimid27 jimid27 merged commit cfc2cf9 into main Mar 26, 2024
@jimid27 jimid27 deleted the chore/add-dedicated-downstream-job-trigger-steps branch March 26, 2024 16:12
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.

2 participants