Skip to content

Conversation

@nrutman
Copy link
Contributor

@nrutman nrutman commented Jun 23, 2025

Overview

Our internal packages configure the @fieldguide namespace to pull from Github for packages like those in https://github.com/fieldguide/js-common.

This causes problems when we also want to access public packages under this namespace. It leads to strange situations like this Action Server NPM configuration.

By publishing the Pipeline package to both NPM and Github Packages, it should allow other Fieldguide repos (e.g. Action Server) to successfully pull the Pipeline package from the Github repository instead of the NPM repository.

with:
node-version-file: ".nvmrc"
cache: "npm"
registry-url: "https://npm.pkg.github.com"

Choose a reason for hiding this comment

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

I wonder if we need to specify scope here, too?

Copy link
Contributor Author

@nrutman nrutman Jun 23, 2025

Choose a reason for hiding this comment

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

I was taking my inspiration from the publish jobs in the js-common repo...for example:

https://github.com/Fieldguide/js-common/blob/main/.github/workflows/publish-files.yaml

...and those don't use a scope when publishing to GitHub.

@nrutman
Copy link
Contributor Author

nrutman commented Jun 23, 2025

For other reviewers…some conversation for context was happening in this Slack thread: https://fieldguideco.slack.com/archives/C018HLNEFF1/p1750707625744909

@nrutman nrutman marked this pull request as ready for review June 23, 2025 20:18
@nrutman nrutman merged commit f9541bb into main Jun 23, 2025
3 checks passed
@nrutman nrutman deleted the naterutman/sc-58776/publish-pipeline-package-to-github branch June 23, 2025 20:18
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