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

Pitchfork gen #217

Merged
merged 29 commits into from
Mar 11, 2024
Merged

Pitchfork gen #217

merged 29 commits into from
Mar 11, 2024

Conversation

mvandeberg
Copy link
Member

Brief description

Generates koinos-proto-cpp in adherence with the pitchfork specification. Also separates build targets in to individual travis builds defined entirely within .travis.yml.

Checklist

  • I have built this pull request locally
  • I have ran the unit tests locally
  • I have manually tested this pull request
  • I have reviewed my pull request
  • I have added any relevant tests

Demonstration

mvandeberg and others added 29 commits March 5, 2024 17:35
@mvandeberg
Put generated protobuf files in correct directories for pitchfork
cd68878
@mvandeberg
Copy source files to src
b856a22
@mvandeberg
Bump protobuf version
d0110a9
@mvandeberg
Go install golang modules
05b38d2
@mvandeberg
Fix rsync include
9fba1fb
@mvandeberg
Update git add to add new dirs
dda93b3
@mvandeberg
Add trailing slash to src directory
5d8d6da
@mvandeberg
Do not exclude for cpp
d76a45a
@mvandeberg
Run C++ as a separate job
f7c273a
@mvandeberg
Fix travis.yml
7ca71cf
@mvandeberg
ccp-out -> cpp_out
170d9f4
@mvandeberg
Combine all as after_success
0f3b0ce
@mvandeberg
Added embedded proto and golang
c665f47
@mvandeberg
Add remaining targets
36d7b3d
@mvandeberg
Cleanup repeated code
fac9dc6
@mvandeberg
Remove ci dir and only run after_success if build is not a PR
8fd1bd1
@mvandeberg
Fix PROTO_FILE->PROTO_FILES
4aab0e5
@mvandeberg
Call protoc from PATH
b34ca25
@mvandeberg
Fix golang versions
73302ac
@mvandeberg
AS pushd
5a4d3d9
@mvandeberg
mkdir -p
10f6716
@mvandeberg
Try embedded proto with PROTO_FILES
935a825
@mvandeberg
Try embedded proto not in embedded proto repo
c4f577d
@mvandeberg
Revert embedded proto changes
f7a6b85
@mvandeberg
Try different PROTO_FILES for embedded proto
3f4a203
@mvandeberg
Find embedded proto files in correct directory
626f994
@mvandeberg
Try to properly setup embedded proto venv by installing venv
d3392a9
@mvandeberg
Error in setup.sh
f27fbba
@sgerbino
Clean up Travis YAML
e2eaefb
@mvandeberg mvandeberg requested a review from sgerbino March 11, 2024 15:58
@sgerbino sgerbino merged commit 5df24f9 into master Mar 11, 2024
2 checks passed
@mvandeberg mvandeberg deleted the pitchfork-gen branch March 11, 2024 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sgerbino sgerbino sgerbino approved these changes

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
@mvandeberg @sgerbino