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: Improve releases #711

Merged
merged 2 commits into from
Nov 17, 2021
Merged

chore: Improve releases #711

merged 2 commits into from
Nov 17, 2021

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 👍

@@ -92,7 +94,7 @@
"react-native-web": "~0.12.0",
"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

@@ -40,7 +40,7 @@ default config for android apk builds: &android_defaults
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.

2 participants