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

Gitpod - add extensions + custom image to get quarkus CLI + contributing.md #25452

Merged
merged 2 commits into from
May 11, 2022

Conversation

jeanphi-baconnais
Copy link
Contributor

Hi,

Using GitPod, I've just changed few small things :

  • add extensions in .gitpod.yml.
  • add Dockerfile to get quarkusCLI available in this project
  • add information in contributing.md

What do you think about these changes ?

Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

Not familiar with GitPod but I added a couple of suggestions.

CONTRIBUTING.md Outdated Show resolved Hide resolved
.gitpod.yml Outdated Show resolved Hide resolved
.gitpod/Dockerfile Outdated Show resolved Hide resolved
@jeanphi-baconnais jeanphi-baconnais force-pushed the gitpod-complete-config branch from aafc55c to fd71e2d Compare May 9, 2022 13:48
Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

I spotted a few minor things. Sorry for not spotting them the first time.

Can you adjust and squash your commits?

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@jeanphi-baconnais jeanphi-baconnais force-pushed the gitpod-complete-config branch from ea33aad to 308a4a8 Compare May 9, 2022 14:22
@jeanphi-baconnais
Copy link
Contributor Author

Thanks for your review and help @gsmet 🙏

@gsmet gsmet force-pushed the gitpod-complete-config branch from 308a4a8 to 8201c4c Compare May 11, 2022 12:44
@gsmet gsmet added the triage/waiting-for-ci Ready to merge when CI successfully finishes label May 11, 2022
@gsmet gsmet changed the title GitPod - add extensions + custom image to get quarkus CLI + contributing.md Gitpod - add extensions + custom image to get quarkus CLI + contributing.md May 11, 2022
@gastaldi gastaldi merged commit 277df2a into quarkusio:main May 11, 2022
@quarkus-bot quarkus-bot bot added this to the 2.10 - main milestone May 11, 2022
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label May 11, 2022
@gastaldi
Copy link
Contributor

Would it make sense to add a button in the README (or even in the CONTRIBUTING guide instead) to open in GitPod?

Open in Gitpod

[![Open in Gitpod](https://gitpod.io/button/open-in-gitpod.svg)](https://gitpod.io/#https://github.com/quarkusio/quarkus/-/tree/main/)

@jeanphi-baconnais
Copy link
Contributor Author

Hi @gastaldi, I was hesitant to add this button 😅 Tried to add this one under badges in the README.md file but wasn't sure if this was the right place. And in the contributor file, there is so much information so 🤷‍♂️ but I could add this one

@jeanphi-baconnais jeanphi-baconnais deleted the gitpod-complete-config branch May 12, 2022 06:54
FROM gitpod/workspace-java-11

RUN bash -c ". /home/gitpod/.sdkman/bin/sdkman-init.sh && \
sdk install java 11.0.9.j9-adpt && \
Copy link
Member

Choose a reason for hiding this comment

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

why use j9 here and not adoptium which we use everywhere else ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh ! I didn't see that. After reading the project, I didn't find the image to use. Where I can find it please ?

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