forked from flyteorg/flyte
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Monorepo import boilerplate #11
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This updates the boilerplate to use go modules instead of dep and follows the changes that have already been made: * flyteorg/datacatalog#21 * flyteorg/flytepropeller#38 * flyteorg/flyteadmin#35 * flyteorg/flyteplugins#36 * flyteorg/flytestdlib#50 * flyteorg/flyteidl#27 (depends on flyteorg/flytestdlib#50 to be merged and released)
bump go version to 1.13
Installing the tools updates the files - let's use the updated files instead
* PR template for all Flyte repos
`;` could hide problems, and it is better to fail fast
* Add github_actions boilerplate * Fix numbering * Parameterize actions further to support forks * remove dockerhub push
* Updating GH workflows to not use the same hard-coded package name across all repos * doc update * Updating readme with a note about forks
* Welcome Bot Message Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Modified update.sh -- in sync with flyteorg/boilerplate repo Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Added .gitignore file Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Modified .gitignore file Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Modified the PR message Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Modified the PR message Signed-off-by: samhita-alla <aallasamhita@gmail.com>
This addresses a TODO in `update.sh` to allow a more freeform `update.cfg`. With this change, `update.cfg` will: - Tolerate and ignore blank/whitespace-only lines - Tolerate `#` style comments, both inline (as long as preceded by whitespace) and whole-line. - Validate directories more prettily, failing with a useful message if one doesn't exist. - Explicitly disallow multiple entries per line, with a useful message. (Previously these last two things would still error, just more confusingly.) While in there, I also added an `EXIT` trap to remove the temporary directory, since otherwise failures mid-script would leave it hanging around, gradually polluting `/tmp`. Signed-off-by: Eric Fried <efried@redhat.com>
* Welcome Bot Message Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Modified update.sh -- in sync with flyteorg/boilerplate repo Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Added .gitignore file Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Modified .gitignore file Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Modified the PR message Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Modified the PR message Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Update Boilerplate Signed-off-by: Samhita Alla <aallasamhita@gmail.com> * Updated permissions of update.sh file Signed-off-by: Samhita Alla <aallasamhita@gmail.com> * Updated update.sh Signed-off-by: Samhita Alla <aallasamhita@gmail.com>
* Welcome Bot Message Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Modified update.sh -- in sync with flyteorg/boilerplate repo Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Added .gitignore file Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Modified .gitignore file Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Modified the PR message Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Modified the PR message Signed-off-by: samhita-alla <aallasamhita@gmail.com> * Update Boilerplate Signed-off-by: Samhita Alla <aallasamhita@gmail.com> * Updated permissions of update.sh file Signed-off-by: Samhita Alla <aallasamhita@gmail.com> * Updated update.sh Signed-off-by: Samhita Alla <aallasamhita@gmail.com> * Modified config.yml path in update.sh Signed-off-by: Samhita Alla <aallasamhita@gmail.com>
Signed-off-by: wild-endeavor <wild-endeavor@users.noreply.github.com>
Signed-off-by: Yuvraj <evalsocket@gmail.com>
* Update update.sh script after everything is done Signed-off-by: Yuvraj <evalsocket@gmail.com> * Added end2end in boilerplate Signed-off-by: Yuvraj <evalsocket@gmail.com> * More changes Signed-off-by: Yuvraj <evalsocket@gmail.com> * More changes Signed-off-by: Yuvraj <evalsocket@gmail.com>
Signed-off-by: Yuvraj <evalsocket@gmail.com>
Signed-off-by: Yuvraj <evalsocket@gmail.com>
Signed-off-by: Yuvraj <evalsocket@gmail.com>
* Added protoc-gen-doc dependency for IDL doc generation Signed-off-by: Prafulla Mahindrakar <prafulla.mahindrakar@gmail.com> * Formatting change Signed-off-by: Prafulla Mahindrakar <prafulla.mahindrakar@gmail.com>
Signed-off-by: Prafulla Mahindrakar <prafulla.mahindrakar@gmail.com>
* Added boilerplate automation Signed-off-by: Yuvraj <code@evalsocket.dev>
Signed-off-by: Yuvraj <code@evalsocket.dev>
…g#33) * Update PR Template to automatically close issues when merged Signed-off-by: Haytham Abuelfutuh <haytham@afutuh.com> * Add instructions to know when to remove 'fixes' keyword Signed-off-by: Haytham Abuelfutuh <haytham@afutuh.com>
Signed-off-by: Yuvraj <code@evalsocket.dev>
* revert go version in support tools Signed-off-by: Yuvraj <code@evalsocket.dev>
Signed-off-by: Haytham Abuelfutuh <haytham@afutuh.com>
Signed-off-by: Yuvraj <code@evalsocket.dev>
Signed-off-by: Prafulla Mahindrakar <prafulla.mahindrakar@gmail.com>
…lag (flyteorg#54) * Use click to define parameters to functional tests script Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com> * Modify invocation of script to add boolean flag Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com> * Revert "Modify invocation of script to add boolean flag" This reverts commit 0f74a78d5b768649dbdf047b83327fe20a361966. Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com> * Modify invocation of functional test script Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com> Co-authored-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>
Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com> Co-authored-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>
Signed-off-by: Prafulla Mahindrakar <prafulla.mahindrakar@gmail.com>
…org#57) Signed-off-by: Prafulla Mahindrakar <prafulla.mahindrakar@gmail.com>
Signed-off-by: Haytham Abuelfutuh <haytham@afutuh.com>
Signed-off-by: Yee Hing Tong <wild-endeavor@users.noreply.github.com>
Signed-off-by: Haytham Abuelfutuh <haytham@afutuh.com>
Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com> Co-authored-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>
Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com> Co-authored-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>
Signed-off-by: Yuvraj <evalsocket@users.noreply.github.com> Co-authored-by: Yuvraj <evalsocket@users.noreply.github.com>
Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com> Co-authored-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>
Signed-off-by: Yuvraj <evalsocket@users.noreply.github.com> Co-authored-by: Yuvraj <evalsocket@users.noreply.github.com>
Signed-off-by: Yuvraj <evalsocket@users.noreply.github.com>
* update perms of code_of_conduct update.sh Signed-off-by: Samhita Alla <aallasamhita@gmail.com> * Remove welcome-bot Signed-off-by: Samhita Alla <aallasamhita@gmail.com>
…ctional OSS testing [ and opta/deployment, etc ] (flyteorg#67) Signed-off-by: Bruce Arctor <5032356+brucearctor@users.noreply.github.com>
* update workflow name Signed-off-by: Kevin Su <pingsutw@apache.org> * update workflow name Signed-off-by: Kevin Su <pingsutw@apache.org> Signed-off-by: Kevin Su <pingsutw@apache.org>
* It will add a lite version of the functional test for quick testing Large functional test good for end 2 end test of releases, but for quick regression test we don't need to validate all workflows, Few workflows will work fine Signed-off-by: Yuvraj <evalsocket@users.noreply.github.com>
Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com> Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com> Co-authored-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>
flyteorg#72) Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com> Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com> Signed-off-by: Eduardo Apolinario <653394+eapolinario@users.noreply.github.com> Co-authored-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>
* Update functional tests to reflect the new names Signed-off-by: eduardo apolinario <eapolinario@users.noreply.github.com> * Fix lite tests Signed-off-by: eduardo apolinario <eapolinario@users.noreply.github.com> --------- Signed-off-by: eduardo apolinario <eapolinario@users.noreply.github.com> Co-authored-by: eduardo apolinario <eapolinario@users.noreply.github.com>
* Fix name of non-P0 tests Signed-off-by: eduardo apolinario <eapolinario@users.noreply.github.com> * Comment pandera test again Signed-off-by: eduardo apolinario <eapolinario@users.noreply.github.com> --------- Signed-off-by: eduardo apolinario <eapolinario@users.noreply.github.com> Co-authored-by: eduardo apolinario <eapolinario@users.noreply.github.com>
Signed-off-by: Daniel Rammer <daniel@union.ai>
…into monorepo--import-boilerplate Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Import boilerplate