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

image addon fixes #4631

Merged
merged 6 commits into from Aug 1, 2023
Merged

image addon fixes #4631

merged 6 commits into from Aug 1, 2023

Conversation

jerch
Copy link
Member

@jerch jerch commented Aug 1, 2023

TODO:

  • remaining changes from 0.4.3 (defer canvas creation & cleanup lifecycling)
  • adjust README
  • fix flakey tests

Fixes #4620, fixes #4627.

@jerch jerch marked this pull request as ready for review August 1, 2023 15:42
@jerch
Copy link
Member Author

jerch commented Aug 1, 2023

@Tyriar The last commit adds the image addon to the publish script. I am not sure if this will work out of the box or if this will create account access issues on npm.

@Tyriar Tyriar self-assigned this Aug 1, 2023
@Tyriar Tyriar added this to the 5.3.0 milestone Aug 1, 2023
@Tyriar Tyriar merged commit c3def09 into xtermjs:master Aug 1, 2023
8 checks passed
@Tyriar
Copy link
Member

Tyriar commented Aug 1, 2023

@jerch looks like it did fail, I'm locked out of my npm account currently but it might just be a matter of you giving collaborator access to either my account or an xterm.js one if we set that up

@jerch
Copy link
Member Author

jerch commented Aug 1, 2023

I can add you, if I find out how to do that 😮

Gimme 5 minutes to mess around with npm account stuff...

@jerch
Copy link
Member Author

jerch commented Aug 1, 2023

Sent you an invitation. Not sure what you have to do to complete it though...

@Tyriar
Copy link
Member

Tyriar commented Aug 1, 2023

Got the invitation, can't use it until I can login though (it said 1-3 days)

@jerch
Copy link
Member Author

jerch commented Aug 1, 2023

Eww, is that a security concept? Let ppl wait long enough to shake off the bad guys? 🤣

@Tyriar
Copy link
Member

Tyriar commented Aug 2, 2023

@jerch my 2FA recovery codes aren't working, so I think it needs a manual check

@jerch
Copy link
Member Author

jerch commented Aug 2, 2023

Just checked the package settings again on npm - looks like you have maintainer state now. So maybe it just works now?

@Tyriar
Copy link
Member

Tyriar commented Aug 2, 2023

@jerch yep got into the account. I re-ran the job and it succeeded 🎉 https://dev.azure.com/xtermjs/xterm.js/_build/results?buildId=6493&view=logs&j=15c39577-a302-5fb0-b018-53da76770408&t=510866e4-c5fa-511e-f61e-bee9c6d19f98

I see it also re-published a slightly older build for xterm, but that'll be sorted out on next merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants