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: set up proper publishing pipeline #2266

Merged
merged 10 commits into from
Dec 4, 2024
Merged

chore: set up proper publishing pipeline #2266

merged 10 commits into from
Dec 4, 2024

Conversation

tido64
Copy link
Member

@tido64 tido64 commented Nov 1, 2024

Summary:

Sets up a publishing pipeline with Nx Release.

Note that while this does enable us to publish nightlies, it will be handled separately from this PR.

  • Replace Beachball with Nx
  • Update integration pipelines
  • Update publish pipeline
  • Point nightly tag to latest nightly build (npm)
  • Create next tag and point it to latest build (npm)

Test Plan:

Follow the instructions in the releases docs and verify that they work for you: https://github.com/microsoft/react-native-macos/blob/cd4104ac8fbca08d60829cef56eed66b56b7d49d/docs/Releases.md

@tido64 tido64 requested a review from a team as a code owner November 1, 2024 14:16
@tido64 tido64 marked this pull request as draft November 1, 2024 14:16
@tido64 tido64 force-pushed the tido/fix-publish branch 2 times, most recently from a9fe079 to c730181 Compare November 1, 2024 14:26
package.json Outdated Show resolved Hide resolved
nx.json Show resolved Hide resolved
.ado/scripts/prepublish-check.mjs Outdated Show resolved Hide resolved
.ado/scripts/prepublish-check.mjs Outdated Show resolved Hide resolved
.ado/scripts/prepublish-check.mjs Outdated Show resolved Hide resolved
@Saadnajmi Saadnajmi marked this pull request as ready for review November 2, 2024 02:40
@tido64 tido64 marked this pull request as draft November 4, 2024 12:03
@tido64 tido64 force-pushed the tido/fix-publish branch 5 times, most recently from a335e87 to e4e70f3 Compare November 7, 2024 12:59
@tido64 tido64 force-pushed the tido/fix-publish branch 12 times, most recently from ebed406 to 2a1da0e Compare November 26, 2024 12:33
@tido64 tido64 marked this pull request as ready for review November 26, 2024 12:42
packages/react-native/package.json Show resolved Hide resolved
nx.json Show resolved Hide resolved
.ado/templates/npm-publish.yml Outdated Show resolved Hide resolved
.ado/templates/npm-publish.yml Show resolved Hide resolved
nx.json Show resolved Hide resolved
Copy link
Collaborator

@Saadnajmi Saadnajmi left a comment

Choose a reason for hiding this comment

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

Thank you for this! Last ask, could we update this guide and/or write a new guide for how releases run / how to test releases locally? https://github.com/microsoft/react-native-macos/blob/main/docs/Releases.md

@tido64
Copy link
Member Author

tido64 commented Nov 28, 2024

@Saadnajmi: Do you have access to the package in npm? We need to create/fix two tags, next and nightly, for all of this to work:

image

(See PR description)

Once they are fixed, this check should be green:

image

@tido64 tido64 force-pushed the tido/fix-publish branch 3 times, most recently from 51c59b5 to cd4104a Compare November 28, 2024 11:34
@Saadnajmi
Copy link
Collaborator

@Saadnajmi: Do you have access to the package in npm? We need to create/fix two tags, next and nightly, for all of this to work:

image

(See PR description)

Once they are fixed, this check should be green:

image

I do, I'll look into that

@Saadnajmi Saadnajmi merged commit dde6d80 into main Dec 4, 2024
14 checks passed
@Saadnajmi Saadnajmi deleted the tido/fix-publish branch December 4, 2024 18:01
Saadnajmi added a commit to Saadnajmi/react-native-macos that referenced this pull request Dec 4, 2024
* chore: set up proper publishing pipeline

* simplify configuration and added docs

* more docs

* add TODO for nightlies

* typos

* also verify ADO pipeline

* refactor checks

* docs

---------

Co-authored-by: Saad Najmi <[email protected]>
Saadnajmi added a commit that referenced this pull request Dec 4, 2024
* chore: set up proper publishing pipeline (#2266)

* chore: set up proper publishing pipeline

* simplify configuration and added docs

* more docs

* add TODO for nightlies

* typos

* also verify ADO pipeline

* refactor checks

* docs

---------

Co-authored-by: Saad Najmi <[email protected]>

* Update publish config

* Fixup versions across monorepo

* Add a version plan file to patch RNM

* pre-run `set-rn-artifacts-version.js` sicne nx-release will not.

---------

Co-authored-by: Tommy Nguyen <[email protected]>
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