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

Try to fix security alert #1429

Merged
merged 1 commit into from Aug 7, 2019
Merged

Conversation

liuzhe-lz
Copy link
Contributor

In PR #1412 , I removed most resolutions after upgrading dependencies.
Seems one of them is still needed.

@liuzhe-lz liuzhe-lz merged commit f721b43 into microsoft:master Aug 7, 2019
liuzhe-lz added a commit that referenced this pull request Aug 7, 2019
liuzhe-lz pushed a commit that referenced this pull request Aug 14, 2019
* Update filter description and fix typo

* fix comments

* change node to result

* Add SupportTrainingService.md (#1401)

* fix nnictl schema

* Eject from react-scripts-ts-antd and bump webui dependencies version (#1412)

* Eject from react-scripts-ts-antd

* test whether it can pass CI without ugilfy

* temporarily disable uglify

* Try to fix security alert (#1429)

* fix bug of hyper-parameter broken when have not succeeded trial

* update

* update
liuzhe-lz pushed a commit that referenced this pull request Aug 20, 2019
…lt (#1472)

* Update filter description and fix typo

* fix comments

* change node to result

* Add SupportTrainingService.md (#1401)

* fix nnictl schema

* Eject from react-scripts-ts-antd and bump webui dependencies version (#1412)

* Eject from react-scripts-ts-antd

* test whether it can pass CI without ugilfy

* temporarily disable uglify

* Try to fix security alert (#1429)

* fix bug of detail page broken when trial is succeed but not report final result
@liuzhe-lz liuzhe-lz deleted the fix-security-alert branch October 9, 2019 04:02
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.

None yet

3 participants