Skip to content

Conversation

stanthetiger
Copy link
Contributor

Update npm package dependencies to the latest.
Keeping vue-loader the same at 15.9.7.
Needed to modify webpack.config.js to get css-loader 6.2.0 working - see css-loader CHANGELOG

@lafriks
Copy link
Member

lafriks commented Jul 20, 2021

fronted check fails

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 20, 2021
@silverwind
Copy link
Member

There is more work to be done here, like new eslint rule additions, svg rebuild, eslint rule additions, lint error fixes and possibly more stuff based on the changelogs of the affected modules. I can probably follow up with a more complete PR soon.

@stanthetiger
Copy link
Contributor Author

I am unable to get eslint to resolve path to module 'monaco-editor' (import/no-unresolved) in features/codeeditor.js

@silverwind
Copy link
Member

silverwind commented Jul 21, 2021

I guess the ESM exports of monaco-editor may have changed or something, I can't find anything in the changelog thought.

@silverwind
Copy link
Member

PR that should solve everything: #16708

silverwind added a commit to silverwind/gitea that referenced this pull request Aug 16, 2021
- Update all JS dependencies
- Adapt to recent webpack changes
- Add new lint rules and fix issues
- Regenerate SVGs and update svgo api usage

Fixes: go-gitea#16492
techknowlogick pushed a commit that referenced this pull request Aug 17, 2021
* Update JS dependencies

- Update all JS dependencies
- Adapt to recent webpack changes
- Add new lint rules and fix issues
- Regenerate SVGs and update svgo api usage

Fixes: #16492

* adapt jest config and sort keys

Co-authored-by: Lunny Xiao <[email protected]>
@go-gitea go-gitea locked and limited conversation to collaborators Oct 19, 2021
@stanthetiger stanthetiger deleted the update-dependencies branch December 31, 2021 09:27
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