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

Aligned to ckeditor5-dev ESM changes #530

Merged
merged 5 commits into from
Oct 3, 2024
Merged

Aligned to ckeditor5-dev ESM changes #530

merged 5 commits into from
Oct 3, 2024

Conversation

psmyrek
Copy link
Contributor

@psmyrek psmyrek commented Sep 19, 2024

Suggested merge commit message (convention)

Internal: Aligned to ckeditor5-dev ESM changes.


Additional information

This PR requires ckeditor/ckeditor5-dev#1011 to be released. When it is released, all ckeditor5-dev-* packages need to be bumped in this repository.

@coveralls
Copy link

coveralls commented Sep 19, 2024

Pull Request Test Coverage Report for Build f01eb93c-0472-4d2c-ad3e-e818b2e0ccb3

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 743f1732-1a9c-4eea-85b0-300aa9e6ce9c: 0.0%
Covered Lines: 584
Relevant Lines: 584

💛 - Coveralls

@pomek
Copy link
Member

pomek commented Sep 23, 2024

I hope we will be able to remove the resolutions key, too:

"resolutions": {
"wrap-ansi": "7.0.0",
"string-width": "4.1.0",
"semver": "^7.0.0",
"ws": "^8"
},

package.json Show resolved Hide resolved
@psmyrek psmyrek marked this pull request as ready for review October 2, 2024 11:56
@pomek pomek merged commit 5f93e09 into master Oct 3, 2024
6 checks passed
@pomek pomek deleted the ci/3772 branch October 3, 2024 05:01
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.

3 participants