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

builds: fix failed to install pnpm #2248

Merged
merged 2 commits into from
Apr 18, 2024
Merged

builds: fix failed to install pnpm #2248

merged 2 commits into from
Apr 18, 2024

Conversation

HSunboy
Copy link
Member

@HSunboy HSunboy commented Apr 18, 2024

What type of PR is this?

Set the pnpm version to v8 to avoid incompatibility with node16

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:


@HSunboy HSunboy changed the base branch from main to dev/4.2.x April 18, 2024 03:22
@yhilmare yhilmare changed the title Fix/xiaokang pnpmp builds: fix failed to install pnpm Apr 18, 2024
@yhilmare yhilmare added this to the ODC 4.2.4-bp2 milestone Apr 18, 2024
@yhilmare
Copy link
Contributor

script/functions.sh needs to be modified also?

Copy link
Collaborator

@tonghan-gsh tonghan-gsh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Contributor

@yhilmare yhilmare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@yhilmare yhilmare merged commit fe69427 into dev/4.2.x Apr 18, 2024
14 checks passed
@yhilmare yhilmare deleted the fix/xiaokang_pnpmp branch April 18, 2024 03:57
zijiacj pushed a commit that referenced this pull request Apr 25, 2024
* fix: set pnpm version to v8

* fix: set pnpm version to v8
zijiacj pushed a commit that referenced this pull request May 7, 2024
* fix: set pnpm version to v8

* fix: set pnpm version to v8
zijiacj pushed a commit that referenced this pull request May 7, 2024
* fix: set pnpm version to v8

* fix: set pnpm version to v8
yhilmare pushed a commit that referenced this pull request May 8, 2024
* fix: set pnpm version to v8

* fix: set pnpm version to v8
yhilmare added a commit that referenced this pull request May 8, 2024
* add serialVersionUID for ConnectionConfig

* Add the sequence version number

* Change the class serialVersionUID to version 4.2.3

* builds: fix failed to install pnpm (#2248)

* fix: set pnpm version to v8

* fix: set pnpm version to v8

* fix approve bug

---------

Co-authored-by: Xiao Kang <xxk268858@antgroup.com>
Co-authored-by: yh263208 <yh263208@oceanbase.com>
PeachThinking pushed a commit that referenced this pull request May 13, 2024
* fix: set pnpm version to v8

* fix: set pnpm version to v8
yhilmare pushed a commit that referenced this pull request May 20, 2024
…e databases change (#2275)

* complete the controller and dao layers

* complete the controller and dao layers

* Complete the code for the template section

* create multiple database change flowinstance

* Added View the details of  multiple database  change flowinstance

* modify code format

* Change the version number to avoid duplication

* add sql check,task log and synchronization of the tasks' progress

* modify code format

* Delete obsolete files

* implement the servic of multiple databases change template

* implement the paging query and multiple databases sqlcheck

* implement the paging query and multiple databases sqlcheck

* Added the duplicate name verification interface and unit test of the template

* modify code format

* builds: fix failed to install pnpm (#2248)

* fix: set pnpm version to v8

* fix: set pnpm version to v8

* Remove redundant code

* Add name for multiple database change flow instance

* modify the code format

* modify view the template list、batch access database、set the risk level and modify the code format

* Add environment objects to viewing  multi-database flow's  response and add database objects for viewing  multi-database sql check's response

* Add a single pre-check node and flow  description for multiple databases

* add the function of viewing multi-database pre-check nodes

* Refine the execution of recorded responses

* Add environment elements to multi-databases response results

* Modify code format

* Limit the number of databases and do not allow duplicate databases

* Fixed error after sql check when the input is '#'

* Solve the problem that multiple database flow cannot be initiated repeatedly

* Fixed NullPointerException  when transferring sql files

* fix execution record display when failure occurs

* modify code format

* enhancement(jpa): add json type (#2211)

* add json type

* remove package-info

* add integrate test

* Modify code format and optimize performance

* Finished ignoring error execution in automatic mode and fixed the issue of checking multiple sql displays

* fix template create and update

* Complete various strategy for creating multi-database workflow

* Reduce redundant code

* Modify the code to conform to odc specifications

* Remove redundant code

* Added a scheduled template task to fix bugs

* modify the code format

* response to cr comments

* Optimize persistence and response objects

* Remove redundant code

* Serialize the databaseList

* modify code format

* delete the redundant code

* modify the code format

* response to cr

* response to cr

---------

Co-authored-by: Xiao Kang <xxk268858@antgroup.com>
Co-authored-by: Ang <43255684+ungreat@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants