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

feat!: add ATLAS_OPTIONS and remove i18n-merge.js | FC-0012 #181

Merged
merged 1 commit into from
Jan 22, 2024

Conversation

OmarIthawi
Copy link
Contributor

This pull request is part of the FC-0012 project which implements the Translation Infrastructure update OEP-58.

@OmarIthawi OmarIthawi changed the title feat!: add ATLAS_OPTIONS and remove i18n-merge.js | FC-0012- feat!: add ATLAS_OPTIONS and remove i18n-merge.js | FC-0012 Jan 2, 2024
Copy link
Contributor

@regisb regisb left a comment

Choose a reason for hiding this comment

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

Please update the README instructions and add a changelog entry :)

@OmarIthawi
Copy link
Contributor Author

Please update the README instructions and add a changelog entry :)

I hear that as a "yes" :)

Will do. Thanks @regisb.

@regisb
Copy link
Contributor

regisb commented Jan 8, 2024

Oh wait, this PR should be on top of the nightly branch, since we don't aim to support Quince.

@OmarIthawi
Copy link
Contributor Author

That's right @regisb. I meant to open it on nightly.

@OmarIthawi OmarIthawi changed the base branch from master to nightly January 8, 2024 15:54
@OmarIthawi OmarIthawi force-pushed the atlas-options branch 2 times, most recently from 4ddd935 to d5c186d Compare January 9, 2024 08:21
@OmarIthawi OmarIthawi marked this pull request as ready for review January 9, 2024 08:21
@OmarIthawi OmarIthawi requested a review from regisb January 9, 2024 08:29
@OmarIthawi
Copy link
Contributor Author

@arbrandes @regisb this is now ready.

Copy link
Contributor

@regisb regisb left a comment

Choose a reason for hiding this comment

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

This looks great to me! What do you think @arbrandes?

Copy link
Collaborator

@arbrandes arbrandes left a comment

Choose a reason for hiding this comment

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

This is awesome. Thanks @OmarIthawi!

@OmarIthawi
Copy link
Contributor Author

@arbrandes All notes have been address, this pull request is now ready.

cc: @brian-smith-tcril @regisb

Copy link
Contributor

@regisb regisb left a comment

Choose a reason for hiding this comment

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

LGTM!

@regisb regisb merged commit 1d6c5a9 into overhangio:nightly Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

4 participants