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

Update the URLs for emoji data [FDN-548] #130

Merged
merged 1 commit into from
Jun 26, 2022
Merged

Update the URLs for emoji data [FDN-548] #130

merged 1 commit into from
Jun 26, 2022

Conversation

yob
Copy link
Contributor

@yob yob commented Jun 26, 2022

In August 2021 we renamed the development branch in the emojis repo from master to main.

That was relatively smooth from an emojis development perspective, but we didn't realise the cli had master hard coded.

As a quick 'fix' I've re-pushed a master branch to the emojis repo, based on current main. It won't stay up to date though, so this changes the cli to check the main branch instead.

See also: buildkite/emojis#386

In August 2021 we renamed the development branch in the emojis repo from
master to main.

That was relatively smooth from an emojis development perspective, but
we didn't realise the cli had master hard coded.

As a quick 'fix' I've re-pushed a master branch to the emojis repo,
based on current main. It won't stay up to date though, so this changes
the cli to check the main branch instead.

See also: buildkite/emojis#386
@yob yob requested a review from a team June 26, 2022 07:04
@yob yob changed the title Update the authoritative URLs for emoji data Update the authoritative URLs for emoji data [FDN-548] Jun 26, 2022
@yob yob changed the title Update the authoritative URLs for emoji data [FDN-548] Update the URLs for emoji data [FDN-548] Jun 26, 2022
@yob yob merged commit 505a970 into main Jun 26, 2022
@yob yob deleted the update-emoji-url branch June 26, 2022 09:03
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