Skip to content

Add Jenkinsfile and gitignore#1

Closed
ligangty wants to merge 1 commit into
project-ncl:mainfrom
ligangty:main
Closed

Add Jenkinsfile and gitignore#1
ligangty wants to merge 1 commit into
project-ncl:mainfrom
ligangty:main

Conversation

@ligangty

@ligangty ligangty commented Nov 9, 2022

Copy link
Copy Markdown
Contributor

No description provided.

@rnc

rnc commented Nov 17, 2022

Copy link
Copy Markdown
Collaborator

Is this for some internal Jenkins? Why not setup a GitHub Actions workflow instead?

@rnc rnc mentioned this pull request Nov 17, 2022
@ligangty

Copy link
Copy Markdown
Contributor Author

@rnc We have an internal indy to host all snapshot version with each PR build merging to publish dependency for internal usage. Of course the github action sounds good too, will add later.

@ligangty

Copy link
Copy Markdown
Contributor Author

Oh, just found another PR for github actions. :-)

@rnc

rnc commented Nov 18, 2022

Copy link
Copy Markdown
Collaborator

@rnc We have an internal indy to host all snapshot version with each PR build merging to publish dependency for internal usage. Of course the github action sounds good too, will add later.

What I do for other repos I control is simply publish each snapshot to sonatype snapshots repo via github actions - this removed the need for any internal jenkins/repository hosting/etc. And the snapshots would be published after each PR merge.

@ligangty

Copy link
Copy Markdown
Contributor Author

@rnc We have an internal indy to host all snapshot version with each PR build merging to publish dependency for internal usage. Of course the github action sounds good too, will add later.

What I do for other repos I control is simply publish each snapshot to sonatype snapshots repo via github actions - this removed the need for any internal jenkins/repository hosting/etc. And the snapshots would be published after each PR merge.

Ok. Seems this repo is not like commonjava repos which is mostly used in our internal team. I think the internal build process is not so important. I'll remove the jenkinsfile there.

@ligangty

Copy link
Copy Markdown
Contributor Author

Dropped this PR.

@ligangty ligangty closed this Nov 18, 2022
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.

2 participants