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

Update JS dependencies #12782

Merged
merged 3 commits into from Sep 10, 2020
Merged

Update JS dependencies #12782

merged 3 commits into from Sep 10, 2020

Conversation

silverwind
Copy link
Member

@silverwind silverwind commented Sep 9, 2020

  • Update all dependencies
  • Add explicit postcss dependency as dictated by postcss-loader
  • Adapt for new postcss-loader syntax
  • Move sourceMap options to top for consistency

The Fomantic update seems compatible, the dropdown accessibilty hack is still in effect.

- Update all dependencies
- Add explicit postcss dependency as dictated by postcss-loader
- Adapt for new postcss-loader syntax
- Move sourceMap options to top for consistency
@lafriks lafriks added the type/refactoring Existing code has been cleaned up. There should be no new functionality. label Sep 9, 2020
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Sep 9, 2020
@lafriks lafriks added this to the 1.13.0 milestone Sep 9, 2020
@lafriks lafriks added dependencies and removed type/refactoring Existing code has been cleaned up. There should be no new functionality. labels Sep 9, 2020
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Sep 9, 2020
@lunny
Copy link
Member

lunny commented Sep 10, 2020

make L-G-T-M work.

@lunny lunny merged commit a775428 into go-gitea:master Sep 10, 2020
@silverwind silverwind deleted the deps-15 branch September 10, 2020 04:33
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants