Skip to content

Make GitHub ci#7

Closed
hiergaut wants to merge 121 commits intomainfrom
makeGithubCI
Closed

Make GitHub ci#7
hiergaut wants to merge 121 commits intomainfrom
makeGithubCI

Conversation

@hiergaut
Copy link
Copy Markdown
Contributor

@hiergaut hiergaut commented Mar 12, 2024

  • Make complete CI for linux, windows and macos.
    Full pipeline when pull request on main branch :
    configure -> build -> install -> test -> coverage -> cppcheck -> format -> doc
  • Make full licensing of project and external
  • Check history (remove confidential/large files)

@nmellado
Copy link
Copy Markdown
Contributor

Postponed until the project goes public.

@hiergaut hiergaut force-pushed the makeGithubCI branch 4 times, most recently from 4e63f92 to d2e6a4b Compare March 20, 2024 16:07
@hiergaut
Copy link
Copy Markdown
Contributor Author

After rebasing this branch to separate the different contributions made here and prevent future inter-contribution conflicts.
This branch is ready to be split in this unique order to avoid future conflicts :
cppCheck -> fix tests -> external licensing -> header versioning -> doxygen -> format (cmake,src) -> gitlab ci -> github ci
This branch do not be pushed, it will be deleted after complete splitting done

Copy link
Copy Markdown
Contributor

@nmellado nmellado left a comment

Choose a reason for hiding this comment

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

Post a changes request to prevent merging: the following PR is going to be split (as discussed AFK).
In the meantime, it is kept open for change tracking.

@hiergaut hiergaut marked this pull request as draft April 4, 2024 08:34
@hiergaut
Copy link
Copy Markdown
Contributor Author

@nmellado Can I close this splitted PR into (#7, #20, #17, #15, #21, #26, #27, #42) ?

@nmellado
Copy link
Copy Markdown
Contributor

Sure, closing.

@nmellado nmellado closed this Apr 22, 2024
@hiergaut hiergaut deleted the makeGithubCI branch April 22, 2024 15:57
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