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

Prep for Diva6 release #406

Closed
55 of 58 tasks
EricHanLiu opened this issue Jul 18, 2018 · 12 comments
Closed
55 of 58 tasks

Prep for Diva6 release #406

EricHanLiu opened this issue Jul 18, 2018 · 12 comments

Comments

@EricHanLiu
Copy link
Contributor

EricHanLiu commented Jul 18, 2018

To-do list

PR #437 contains a lot of changes, accepted!

@EricHanLiu
Copy link
Contributor Author

EricHanLiu commented Jul 18, 2018

Fixed JSHint syntax issues and added proper Babel preset for Webpack compiling

@EricHanLiu EricHanLiu added this to the RELEASE-6.0 milestone Jul 18, 2018
@EricHanLiu
Copy link
Contributor Author

EricHanLiu commented Jul 18, 2018

None of the unit tests work on master or develop-diva6

edit: they work now with some tinkering

@EricHanLiu
Copy link
Contributor Author

Dependencies updated

gulp -> 4.0.0
webpack -> 4.14.0
webpack-dev-server -> 3.1.4
mocha -> 5.2.0
babel-loader -> 7.1.4

@ahankinson
Copy link
Member

As this release is imminent:

  1. Merge your commits on this branch back to develop-diva6
  2. Merge those commits from develop-diva6 to develop
  3. Base your changes going forward on the develop branch.

There is no need to keep so many branches around.

@EricHanLiu
Copy link
Contributor Author

Everything merged into develop

@ahankinson
Copy link
Member

OK. You should probably break out the other checkboxed to-dos into specific issues.

@EricHanLiu
Copy link
Contributor Author

reminder to fix event listener issue on fullscreen mode

@EricHanLiu
Copy link
Contributor Author

Issue #434 and #404 are awaiting PR #435

Issue #375 is being done on the filter queue branch

@EricHanLiu
Copy link
Contributor Author

Release should be ready

@EricHanLiu
Copy link
Contributor Author

pr #466

@EricHanLiu
Copy link
Contributor Author

This isn't being automatically added to CDNJS based on the git release tag (which it says it should be doing)

@EricHanLiu
Copy link
Contributor Author

Closing this in favour of the existing open issues for CDNJS and cantus ultimus testing. Other issues will be relative to a new minor release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment