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

added basic CI #10

Merged
merged 2 commits into from
Nov 15, 2023
Merged

added basic CI #10

merged 2 commits into from
Nov 15, 2023

Conversation

g-arakelov
Copy link
Contributor

@g-arakelov g-arakelov commented Oct 30, 2023 •
edited by NickVolynkin
Loading

@NickVolynkin NickVolynkin force-pushed the 8-add-ci branch 2 times, most recently from bddfc81 to 4c2d6ad Compare November 15, 2023 15:49
NickVolynkin
NickVolynkin approved these changes Nov 15, 2023
View reviewed changes
Copy link
Contributor

@NickVolynkin NickVolynkin left a comment

Choose a reason for hiding this comment

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

I've turned off the set_version workflow because I have doubts about it. Otherwise, LGTM. Thank you!

@NickVolynkin
Copy link
Contributor

Don't merge. I'll join the workflows.

@NickVolynkin
ci: build deb always, push on tags
3d50d89 
* Build .deb packages on every CI run.
* Push them only when tags are pushed.
  Note: CI doesn't check that tagged commit belongs to a stable branch.
  Check it yourself.
* Update readme.md with info about .deb releases.

Remove the auto-tagging CI.
We don't need that many tags, because not every commit in master gets
a proper integration testing.
@NickVolynkin NickVolynkin merged commit af1e5bf into master Nov 15, 2023
2 checks passed
@NickVolynkin NickVolynkin deleted the 8-add-ci branch November 15, 2023 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NickVolynkin NickVolynkin NickVolynkin approved these changes

@SK0M0R0H SK0M0R0H Awaiting requested review from SK0M0R0H

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@g-arakelov @NickVolynkin