Skip to content
This repository was archived by the owner on Oct 28, 2024. It is now read-only.

Dotenv vault - #244

Open
StudioCordillera wants to merge 9 commits into
alibaba:mainfrom
StudioCordillera:dotenv-vault
Open

Dotenv vault#244
StudioCordillera wants to merge 9 commits into
alibaba:mainfrom
StudioCordillera:dotenv-vault

Conversation

@StudioCordillera

Copy link
Copy Markdown

No description provided.

@CLAassistant

CLAassistant commented May 29, 2023

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ StudioCordillera
❌ dotenv-org


dotenv-org seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@@ -0,0 +1,222 @@
# don't import any costly modules

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Files under the venv directory belong to the Python development environment and should not be committed.

Comment thread ali-dbhub-server/pom.xml
<groupId>cn.hutool</groupId>
<artifactId>hutool-all</artifactId>
<version>5.8.5</version>
<version>5.8.18</version>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For PRs that modify documentation, dependency version numbers should not be changed. If a change in dependency version number is required, a separate PR should be opened for it and tests need to be conducted.

* There will be problems with the cross -domain of the CORSRegistry strategy, but there is no problem in the local area, but there is no local recovery. The reason may be: the problem of the loading order of the bean.
* Temporary solution through CORSFILTER, follow -up can be studied: Corsregistry
*
* @Author is the instrument

@yuanzhixiang yuanzhixiang May 29, 2023

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Translate '是仪' as 'shiyi', not 'is the instrument'.

"resolveJsonModule": true,
"importHelpers": true,
"jsx": "react",
"jsx": "react-jsx",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For PRs that modify the documentation, dependencies should not be changed. If a change in dependencies is required, a separate PR should be opened for it and testing must be conducted.

"react": "17.x",
"react-dom": "17.x",
"react": "^16.14.0",
"react-dom": "^16.14.0",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For PRs that modify documentation, dependency version numbers should not be changed. If a change in dependency version number is required, a separate PR should be opened for it and tests need to be conducted.

Comment thread .env.vault
@@ -0,0 +1,21 @@
#/-------------------.env.vault---------------------/

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This file should not be committed.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants