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.

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

feat: Run Flank Github action with config provided by user #1673

Merged
merged 6 commits into from
Mar 10, 2021

Conversation

piotradamczyk5
Copy link
Contributor

@piotradamczyk5 piotradamczyk5 commented Mar 9, 2021

Fixes #1598

Test Plan

How do we know the code works?

Flank Github action works properly 🎉
Please see the test repository for running examples

Checklist

  • Fix reading service account
  • Add options to pass flank config

@piotradamczyk5 piotradamczyk5 changed the title #1598 added reading config file feat: Run Flank Github action with config Mar 9, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Mar 9, 2021

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@piotradamczyk5 piotradamczyk5 changed the title feat: Run Flank Github action with config feat: Run Flank Github action with config provided by user Mar 9, 2021
@bootstraponline bootstraponline force-pushed the #1598_added_reading_config_file branch from d75d21c to 47dea64 Compare March 9, 2021 16:11
@github-actions
Copy link
Contributor

github-actions bot commented Mar 9, 2021

Timestamp: 2021-03-09 17:22:30
Buildscan url for ubuntu-workflow run 636564594
https://gradle.com/s/oh7bdmost3h3o

@piotradamczyk5 piotradamczyk5 force-pushed the #1598_added_reading_config_file branch from 6f06bb2 to 810cac1 Compare March 9, 2021 17:17
runs:
using: "composite"
steps:
- name: Download flankScripts
run: |
curl -L https://github.com/Flank/flank/releases/download/flank-scripts-1.9.0/flank-scripts.jar --output flank-scripts.jar
curl -L https://github.com/Flank/flank/releases/download/flank-scripts-1.9.1/flank-scripts.jar --output flank-scripts.jar
Copy link
Contributor

Choose a reason for hiding this comment

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

is there a possibility to always retrieve the latest?
Therefore we dont need to constantly change the version number?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Sloox
It is by design because we are sure that we will use a stable version tested with this Github action

@Sloox Sloox self-requested a review March 10, 2021 12:53
Copy link
Contributor

@Sloox Sloox left a comment

Choose a reason for hiding this comment

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

Minor comment

@Sloox Sloox self-requested a review March 10, 2021 13:14
@piotradamczyk5 piotradamczyk5 merged commit bfea6a2 into master Mar 10, 2021
@piotradamczyk5 piotradamczyk5 deleted the #1598_added_reading_config_file branch March 10, 2021 13:14
@github-actions github-actions bot locked and limited conversation to collaborators Mar 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Run flank with configuration file provided by user
4 participants