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

Move workflow samples to external files #18135

Merged
merged 1 commit into from
Mar 17, 2022

Conversation

mshima
Copy link
Member

@mshima mshima commented Mar 16, 2022

Allow jhipster-bom project to auto pick samples changes.


Please make sure the below checklist is followed for Pull Requests.

When you are still working on the PR, consider converting it to Draft (below reviewers) and adding skip-ci label, you can still see CI build result at your branch.

@mshima mshima force-pushed the skip_ci-workflow_samples branch 2 times, most recently from a742f2f to ffad841 Compare March 16, 2022 20:50
.github/workflows/angular.yml Outdated Show resolved Hide resolved
.github/workflows/react.yml Outdated Show resolved Hide resolved
.github/workflows/vue.yml Outdated Show resolved Hide resolved
@mshima mshima marked this pull request as ready for review March 17, 2022 02:05
@mshima mshima requested a review from DanielFran March 17, 2022 02:06
@DanielFran
Copy link
Member

@mshima Seems good. Is it missing webflux?

@mshima
Copy link
Member Author

mshima commented Mar 17, 2022

I think we should move webflux samples to angular/react/vue workflows

mshima added a commit to Tcharl/generator-jhipster that referenced this pull request Mar 17, 2022
@mshima mshima changed the title Move workflow samples to an external file Move workflow samples to external files Mar 17, 2022
@mshima mshima merged commit 29f2d7f into jhipster:main Mar 17, 2022
@mshima mshima deleted the skip_ci-workflow_samples branch March 17, 2022 22:12
@pascalgrimaud pascalgrimaud added this to the 7.8.0 milestone Mar 27, 2022
ko5tik pushed a commit to ko5tik/generator-jhipster that referenced this pull request May 21, 2022
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.

None yet

3 participants