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

Fix yo doctor errors during build #24

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

AMoo-Miki
Copy link
Collaborator

@AMoo-Miki AMoo-Miki commented Aug 8, 2022

Description

Removes forced resolution of bin-version-check to a higher and incompatible version to avoid yo-doctor errors during build.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Miki <miki@amazon.com>
@AMoo-Miki AMoo-Miki changed the title Fix yo doctor Fix yo doctor errors during build Aug 8, 2022
@AMoo-Miki AMoo-Miki merged commit 3b2ae65 into opensearch-project:main Aug 8, 2022
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 9, 2022
Signed-off-by: Miki <miki@amazon.com>
(cherry picked from commit 3b2ae65)
AMoo-Miki added a commit that referenced this pull request Aug 17, 2022
Signed-off-by: Miki <miki@amazon.com>
(cherry picked from commit 3b2ae65)

Co-authored-by: Miki <miki@amazon.com>
AMoo-Miki added a commit that referenced this pull request Sep 12, 2022
* Avoid using dev-deps in post-install script (#23)

Signed-off-by: Miki <miki@amazon.com>

* Fix yodoctor errors (#24)

Signed-off-by: Miki <miki@amazon.com>

* Add backport workflow (#26)

Signed-off-by: Miki <miki@amazon.com>

* Add code owners and issue templates (#27)

Signed-off-by: Miki <miki@amazon.com>

* Bump node to 14.20.0 (#36)

Signed-off-by: Miki <miki@amazon.com>

* Bump `pacote` as a transitive dependency of `codesandbox` (#38)

`codesandbox` no longer consumes `pacote` and its inclusion as dep is in error.

Signed-off-by: Miki <miki@amazon.com>

* Fix lint errors (#39)

* Lockdown cheerio to rc.8
* SCSS spacing
* Incorrect alias classname

Signed-off-by: Miki <miki@amazon.com>

* Fix test errors (#42)

* Lockdown cheerio to rc.5

Signed-off-by: Miki <miki@amazon.com>

* Bump packages - CVE (#50)

* Bump packages - CVE

Signed-off-by: kaddy645 <xdeskart@amazon.com>

* Bump moment-timezone from 0.5.31 to 0.5.35 (#54)

Bumps [moment-timezone](https://github.com/moment/moment-timezone) from 0.5.31 to 0.5.35.
- [Release notes](https://github.com/moment/moment-timezone/releases)
- [Changelog](https://github.com/moment/moment-timezone/blob/develop/changelog.md)
- [Commits](moment/moment-timezone@0.5.31...0.5.35)

---
updated-dependencies:
- dependency-name: moment-timezone
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump React 16.12.0 to 16.14.0 (#55)

Signed-off-by: kaddy645 <xdeskart@amazon.com>

Signed-off-by: kaddy645 <xdeskart@amazon.com>

* Bump har-validator (#57)

Signed-off-by: Bandini Bhopi <bandinib@amazon.com>

Signed-off-by: Bandini Bhopi <bandinib@amazon.com>

* Fix compile icon script to include license header (#43)

Issue resolved:
#37

Signed-off-by: Bandini Bhopi <bandinib@amazon.com>

Signed-off-by: Bandini Bhopi <bandinib@amazon.com>

Signed-off-by: Miki <miki@amazon.com>
Signed-off-by: kaddy645 <xdeskart@amazon.com>
Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Bandini Bhopi <bandinib@amazon.com>
Co-authored-by: Kartik Desai <xdeskart@amazon.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Bandini <63824432+bandinib-amzn@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants