Skip to content

Conversation

@krizzu
Copy link
Member

@krizzu krizzu commented Nov 17, 2021

Summary

  • Updated semantic-release package version
  • Added settings to generate CHANGELOG file + auto bump package.json (not happening now)

@krizzu krizzu requested a review from tido64 November 17, 2021 09:18
"@semantic-release/git",
{
"assets": ["CHANGELOG.md", "package.json"],
"message": "chore(release): ${nextRelease.version} [skip ci]"
Copy link
Member

Choose a reason for hiding this comment

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

GitHub Actions does not understand skip ci by default. We need to manually check for it. You can have a look at how it was done in react-native-test-app: microsoft/react-native-test-app@c20eaaa

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

@krizzu krizzu Nov 17, 2021

Choose a reason for hiding this comment

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

good one, will apply that never mind then 👍

"react-native-windows": "^0.63.41",
"react-test-renderer": "16.13.1",
"semantic-release": "^17.2.1"
"semantic-release": "^17.4.6"
Copy link
Member

Choose a reason for hiding this comment

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

Should we bump to 18?

Copy link
Member Author

Choose a reason for hiding this comment

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

I did not bump to latest due to node version required (14), while our CI images are at 12

Copy link
Member

Choose a reason for hiding this comment

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

Should we bump Node to 14 (in a separate PR)?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, that'd require more compatibility checks on CI, so ideally separate PR

Comment on lines +77 to +78
"@semantic-release/changelog": "^5.0.1",
"@semantic-release/git": "9.0.0",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"@semantic-release/changelog": "^5.0.1",
"@semantic-release/git": "9.0.0",
"@semantic-release/changelog": "^6.0.1",
"@semantic-release/git": "^10.0.1",

Copy link
Member Author

Choose a reason for hiding this comment

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

Same as above, those require node 14

resource_class: 'medium'
working_directory: ~/async_storage
environment:
- _JAVA_OPTIONS: '-XX:+UnlockExperimentalVMOptions -XX:+UseCGroupMemoryLimitForHeap -Xmx2048m'
Copy link
Member Author

Choose a reason for hiding this comment

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

Apparently this was never needed
https://stackoverflow.com/a/52555836/3510245

@krizzu krizzu merged commit 94b9520 into master Nov 17, 2021
@krizzu krizzu deleted the semantic-release-improv branch November 17, 2021 10:29
lafest pushed a commit to lafest/async-storage that referenced this pull request Nov 22, 2021
* improve releases

* remove java automatic heap detection
@krizzu
Copy link
Member Author

krizzu commented Nov 24, 2021

🎉 This PR is included in version 1.15.12 🎉

The release is available on:

Your semantic-release bot 📦🚀

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