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

build: add a devcontainer.json #1544

Merged
merged 3 commits into from
Jun 29, 2023
Merged

Conversation

Smityz
Copy link
Contributor

@Smityz Smityz commented Jun 23, 2023

Related issue: #1549

Support developing inside a container, ref: https://code.visualstudio.com/docs/devcontainers/containers
So that we can develop and compile Pegasus in a docker environment.
I can successfully compile the code in my WSL2 environment.
image

@cr-gpt
Copy link

cr-gpt bot commented Jun 23, 2023

Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information

acelyc111
acelyc111 previously approved these changes Jun 25, 2023
@acelyc111
Copy link
Member

The "Standardization Lint" is failed https://github.com/apache/incubator-pegasus/actions/runs/5370641850, it's not related to this PR, we can fix it later.

But AFAIK, this PR lacks of an issue link and the new added file is needed to be added to .licenserc.yaml.

@acelyc111 acelyc111 closed this Jun 26, 2023
@acelyc111 acelyc111 reopened this Jun 26, 2023
@acelyc111
Copy link
Member

The "Standardization Lint" is failed https://github.com/apache/incubator-pegasus/actions/runs/5370641850, it's not related to this PR, we can fix it later.

But AFAIK, this PR lacks of an issue link and the new added file is needed to be added to .licenserc.yaml.

@Smityz Resolved, please see the latest checks.

@acelyc111 acelyc111 merged commit df658a7 into apache:master Jun 29, 2023
@empiredan empiredan mentioned this pull request Aug 22, 2023
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.

4 participants