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.

Sign up for GitHub

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

Jump to bottom

DX: build library on GitHub Actions #13

Merged
merged 35 commits into from
Aug 4, 2023
Merged

DX: build library on GitHub Actions #13

merged 35 commits into from
Aug 4, 2023

Conversation

redeboer
Copy link
Member

@redeboer redeboer commented Aug 3, 2023

No description provided.

@redeboer redeboer self-assigned this Aug 3, 2023
redeboer and others added 28 commits August 3, 2023 18:41
@redeboer
DX: hide wget output
8f12f23
@redeboer
FIX: source ROOT before script
590e557
@redeboer
FIX: use bash instead of dash
98ba7b3
FIX: specify bash shell with shell
af0c0ee 
https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsshell
@redeboer
ENH: cache ROOT installation
a87c0f9
@redeboer
ENH: split workflows to add trigger dependencies
464c21b 
https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#onpushpull_requestpull_request_targetpathspaths-ignore
@redeboer
DX: pin GitHub workflows
4c03808
@redeboer
FIX: replace /bin/sh
739a9f6
@redeboer
FIX: run both make statements in same step
adffc89 
REmoves to source `thisroot.sh` twice
@redeboer
DX: build project on MacOS runner
b3360d7
@redeboer
FIX: remove apt install step
ffc6d5f 
Not even needed on Ubuntu, probably
@redeboer
DX: describe build step
5ed5b1f
@redeboer
MAINT: ignore script binaries
009df24
@redeboer
DX: run executables in CI
044ef3c
@redeboer
DX: only build C++ code if relevant files change
7375495
@redeboer
DX: improve job name
786d9af
@redeboer
DX: kill old CI runs on new commits
9a55278
@redeboer
FIX: only replace shell in Ubuntu
3c22a4b
@redeboer
DX: do not fail-fast job matrix
cfc31f9
@redeboer
FIX: correct contains syntax
46732e8
@redeboer
DX: cache build files
82ad42e
@redeboer
FIX: download correct version of ROOT per OS
1eb0e3d
@redeboer
FIX: navigate to scripts directory
de801f8
@redeboer
FIX: install root through brew
67714b4
@redeboer
FIX: navigate to script directory
e102a80
@redeboer
FIX: only source thisroot.sh in Ubuntu
ebbe114
@redeboer
ENH: build multithreaded
96f5262
@redeboer
DX: clean PR caches after merge
3714c1c
@redeboer redeboer marked this pull request as ready for review August 4, 2023 13:57
@redeboer redeboer merged commit 9cf56ab into main Aug 4, 2023
2 checks passed
@redeboer redeboer deleted the cpp-ci branch August 4, 2023 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@redeboer redeboer

Labels
AmpTools
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@redeboer