Skip to content

feat(dart): rely on common changelog #1788

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

Merged
merged 7 commits into from
Jul 21, 2023
Merged

feat(dart): rely on common changelog #1788

merged 7 commits into from
Jul 21, 2023

Conversation

shortcuts
Copy link
Member

@shortcuts shortcuts commented Jul 20, 2023

🧭 What and Why

🎟 JIRA Ticket: https://algolia.atlassian.net/browse/DI-1440

Changes included:

This PR aims at making Dart leverage the common changelog generation process, while still relying on Melos for version bump.

We now generate a changelog per Dart client, and omitting the changelog generation on the Melos side.

Things to fix:

  • get the correct currentVersion from the Dart package
  • update previously generated changelogs to have the same shape everywhere

How to test:

  • Add LOCAL_TEST_DEV=true to your .env file at the root of the repo
  • yarn docker:release

@shortcuts shortcuts self-assigned this Jul 20, 2023
@netlify
Copy link

netlify bot commented Jul 20, 2023

Deploy Preview for api-clients-automation ready!

Name Link
🔨 Latest commit 04a92a0
🔍 Latest deploy log https://app.netlify.com/sites/api-clients-automation/deploys/64b9363797b9480008005ff7
😎 Deploy Preview https://deploy-preview-1788--api-clients-automation.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@algolia-bot
Copy link
Collaborator

algolia-bot commented Jul 20, 2023

✗ The generated branch has been deleted.

If the PR has been merged, you can check the generated code on the main branch.
You can still access the code generated on main via this commit.

@shortcuts shortcuts marked this pull request as ready for review July 20, 2023 12:03
@shortcuts shortcuts requested a review from a team as a code owner July 20, 2023 12:03
@shortcuts shortcuts requested review from aallam and millotp July 20, 2023 12:03
@shortcuts
Copy link
Member Author

do you want to make it retroactive @aallam

@aallam
Copy link
Member

aallam commented Jul 20, 2023

do you want to make it retroactive @aallam

We are still at alpha stage, we can just skip the old changelog content and start from new.

@shortcuts
Copy link
Member Author

do you want to make it retroactive @aallam

We are still at alpha stage, we can just skip the old changelog content and start from new.

true, I'll trigger a new release soon and delete previous content

@shortcuts shortcuts force-pushed the feat/dart-changelog branch from 9925163 to 04a92a0 Compare July 20, 2023 13:27
Copy link
Collaborator

@millotp millotp left a comment

Choose a reason for hiding this comment

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

looks great !

}

await updateChangelog(lang as Language, changelog[lang], current, next);
await updateChangelog(
lang as Language,
Copy link
Collaborator

Choose a reason for hiding this comment

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

could be nice to use something like this to avoid the type conversion

@shortcuts
Copy link
Member Author

shortcuts commented Jul 21, 2023

fixed the version bump in d7f7889 @aallam

(edit: it's normal to see the js changes in the dart changelog, for testing purposes I've directed all changes to the dart client)

@shortcuts shortcuts requested a review from millotp July 21, 2023 09:24
@aallam
Copy link
Member

aallam commented Jul 21, 2023

fixed the version bump in d7f7889 @aallam

Nice! GJ! 🚀

@shortcuts shortcuts merged commit 81ae3c7 into main Jul 21, 2023
@shortcuts shortcuts deleted the feat/dart-changelog branch July 21, 2023 09:32
@shortcuts
Copy link
Member Author

will try it right away!

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.

4 participants