Skip to content

Revert hardcodes #508

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 1 commit into from
Aug 26, 2021
Merged

Revert hardcodes #508

merged 1 commit into from
Aug 26, 2021

Conversation

armanbilge
Copy link
Member

@japgolly Sorry for messing it all up :(

@japgolly
Copy link
Contributor

@armanbilge No problem at all, I'm sorry I wasn't clearer. I seem to be rushing a lot this year, I'll try to slow down a bit next time.

So just so we're on the same page, if we need to do a new 1.x release, we already have to create the tag manually locally, so let's just also add a commit directly to the branch being tagged, to set the version.

@japgolly japgolly merged commit fd71a8c into series/1.x Aug 26, 2021
@japgolly japgolly deleted the topic/revert-hardcodes branch August 26, 2021 06:05
@japgolly
Copy link
Contributor

Also @armanbilge , are you still good to do the release comms?

@armanbilge
Copy link
Member Author

armanbilge commented Aug 26, 2021

so let's just also add a commit directly to the branch being tagged, to set the version

Ahh, so this is what you did, that's why I couldn't find it. Also, I thought you would have published the release notes I drafted when you published. Well, I know now for next time, though hopefully this will be the last time.

are you still good to do the release comms?

I'll cover gitter, discord, scala-users, do you mind tweeting?

@armanbilge
Copy link
Member Author

Also, I thought you would have published the release notes I drafted when you published.

Shoot, these just got overriden 😅

@japgolly
Copy link
Contributor

I haven't looked into how the release notes stuff works, we have release notes now? How do we find them?

do you mind tweeting?

Yeah np, I'll do Twitter & Reddit but I'll wait to see about these release notes first

@armanbilge
Copy link
Member Author

They are here: https://github.com/scala-js/scala-js-dom/releases

I wrote it up all nice, but it seems my idiot merges just now got them overriden by the release drafter. I'll write them up again.

@armanbilge
Copy link
Member Author

@japgolly I drafted release notes, which you should be able to see at https://github.com/scala-js/scala-js-dom/releases. Please edit as you see fit and publish!

@armanbilge
Copy link
Member Author

armanbilge commented Aug 26, 2021

OMG, I am so stupid. I merged your dependabot PR and my release notes got overwritten again. And I still didn't save them!! 😠

@armanbilge
Copy link
Member Author

Third times a charm 😬 Also saved them externally this time, just in case!

@japgolly
Copy link
Contributor

Oh man, I feel your pain :(

@japgolly
Copy link
Contributor

@armanbilge ok mate, release notes published and I did these comms below. I'll leave the other ones to you but we did it! Hehe, bit of stumbling around but it all worked out well in the end, and we'll improve for v2.0. Thanks for your help! I'm enjoying maintaining with you!

@armanbilge
Copy link
Member Author

Exciting!! 😆 yes it's been a real pleasure, thank you so much for including me! I'm learning so much too 😁 Towards v2 🚀

Yup, I'll take care of the other comms!

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