Skip to content

Conversation

zeripath
Copy link
Contributor

@zeripath zeripath commented Nov 6, 2021

Upgrade monaco to 0.30.0 and other javascript dependencies.

Signed-off-by: Andrew Thornton art27@cantab.net

Upgrade monaco to 0.30.0 and other javascript dependencies.

Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath zeripath added this to the 1.16.0 milestone Nov 6, 2021
zeripath and others added 3 commits November 7, 2021 09:36
Signed-off-by: Andrew Thornton <art27@cantab.net>
Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath
Copy link
Contributor Author

zeripath commented Nov 7, 2021

Looks like we may need to back off some more of these dependency changes to get the docker to build

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Nov 7, 2021
Signed-off-by: Andrew Thornton <art27@cantab.net>
Signed-off-by: Andrew Thornton <art27@cantab.net>
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (main@69b61d4). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main   #17574   +/-   ##
=======================================
  Coverage        ?   45.48%           
=======================================
  Files           ?      793           
  Lines           ?    88846           
  Branches        ?        0           
=======================================
  Hits            ?    40415           
  Misses          ?    41916           
  Partials        ?     6515           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 69b61d4...dfe7c39. Read the comment docs.

"eslint-plugin-unicorn": "37.0.1",
"eslint-plugin-vue": "7.19.1",
"jest": "27.3.0",
"eslint-plugin-unicorn": "38.0.0",
Copy link
Member

Choose a reason for hiding this comment

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

Should add the new rules in .eslintrc, I think both can be enabled, e.g. [2].

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My main point here is get monaco-0.30.0 in as soon as possible. Does this definitely build?

Copy link
Member

@silverwind silverwind Nov 10, 2021

Choose a reason for hiding this comment

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

lint-frontend will fail if the new rules fail, I think it's unlikely that it will.

If you just want to upgrade monaco alone, you could only upgrade it alone, e.g. npm install monaco-editor@latest monaco-editor-webpack-plugin@latest on a fresh branch.

@silverwind
Copy link
Member

More complete PR #17611 that also takes care of one breaking change in Monaco.

silverwind added a commit to silverwind/gitea that referenced this pull request Nov 10, 2021
- Update monaco, adapting to breaking changes
- Update dropzone, adapting to breaking changes
- Update linters, fix new issues, disable opinionated stylelint rules
- Rebuild SVGs and images
- Tested Dropzone and Monaco

Replaces: go-gitea#17574
wxiaoguang pushed a commit that referenced this pull request Nov 11, 2021
- Update monaco, adapting to breaking changes
- Update dropzone, adapting to breaking changes
- Update linters, fix new issues, disable opinionated stylelint rules
- Rebuild SVGs and images
- Tested Dropzone and Monaco

Replaces: #17574
@silverwind silverwind closed this Nov 11, 2021
@zeripath zeripath deleted the monaco-0.30.0 branch November 13, 2021 09:46
Chianina pushed a commit to Chianina/gitea that referenced this pull request Mar 28, 2022
- Update monaco, adapting to breaking changes
- Update dropzone, adapting to breaking changes
- Update linters, fix new issues, disable opinionated stylelint rules
- Rebuild SVGs and images
- Tested Dropzone and Monaco

Replaces: go-gitea#17574
@go-gitea go-gitea locked and limited conversation to collaborators Apr 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants