Skip to content

CircleCI: support for intergration branches #567

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 4 commits into from
Jan 20, 2021
Merged

CircleCI: support for intergration branches #567

merged 4 commits into from
Jan 20, 2021

Conversation

rafie
Copy link
Contributor

@rafie rafie commented Jan 19, 2021

No description provided.

@codecov
Copy link

codecov bot commented Jan 19, 2021

Codecov Report

Merging #567 (888379a) into master (ffd4084) will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #567      +/-   ##
==========================================
+ Coverage   72.31%   72.35%   +0.03%     
==========================================
  Files          37       37              
  Lines        5917     5917              
==========================================
+ Hits         4279     4281       +2     
+ Misses       1638     1636       -2     
Impacted Files Coverage Δ
src/tensor.c 74.73% <0.00%> (+0.35%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ffd4084...888379a. Read the comment docs.

@rafie rafie requested a review from DvirDukhan January 19, 2021 15:25
Comment on lines 355 to 359
filters:
branches:
only: master
tags:
only: /^v[0-9].*/
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove on-master-and-version-tags as it is no longer in use, or combine both definitions

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes.

@@ -405,7 +421,7 @@ workflows:
# <<: *on-version-tags
- deploy-snapshot:
<<: *after-platform-builds
<<: *on-master
<<: *on-integ-branch
Copy link
Collaborator

Choose a reason for hiding this comment

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

why do we want integ branch snapshots? will it overwrite existing versions?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

like we get a master snapshot, we may want integration branch snapshots, with corresponding names, like redisai/snapshots/redisai-cpu-onnxruntime.linux-bionic-x64.1.2.tgz.
we can limit ourselves to on-master at the moment.

@rafie rafie requested a review from DvirDukhan January 20, 2021 10:02
@rafie rafie merged commit 80bf0eb into master Jan 20, 2021
@rafie rafie deleted the rafi_circleci1 branch January 20, 2021 10:55
rafie added a commit that referenced this pull request Jan 20, 2021
rafie added a commit that referenced this pull request Jan 20, 2021
@chayim chayim mentioned this pull request Jul 22, 2021
@chayim chayim mentioned this pull request Nov 7, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants