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

CLN, CI: all scripts and configs in one #50

Merged
merged 8 commits into from
Jul 18, 2021

Conversation

Zeroto521
Copy link
Member

@Zeroto521 Zeroto521 commented Jun 11, 2021

This project is used to store plugins information.
So helper script should not place at the top level of the project.
With more helper scripts come up, we better put them in another place.

ci/src/updater.py Outdated Show resolved Hide resolved
ci/src/updater.py Show resolved Hide resolved
.github/workflows/updater.yaml Outdated Show resolved Hide resolved
ci/src/updater.py Outdated Show resolved Hide resolved
.github/workflows/updater.yaml Show resolved Hide resolved
.github/workflows/validator.yaml Outdated Show resolved Hide resolved
ci/src/updater.py Show resolved Hide resolved
ci/src/validator.py Show resolved Hide resolved
ci/src/validator.py Show resolved Hide resolved
@Zeroto521
Copy link
Member Author

It's ready to merge.

@taooceros
Copy link
Member

taooceros commented Jul 16, 2021

@Zeroto521 @jjw24 After the merge of #57 , these pr won't be tested, but the requirement still exist. Do you know how to remove that?

@Zeroto521
Copy link
Member Author

@Zeroto521 @jjw24 After the merge of #57 , these pr won't be tested, but the requirement still exist. Do you know how to remove that?

discuss this at #58

@jjw24
Copy link
Member

jjw24 commented Jul 17, 2021

Sorry I have not been looking at this PR, I think @taooceros you have right? Let me know if it's good to go and I will manually merge it

@jjw24 jjw24 changed the base branch from main to plugin_api_v2 July 18, 2021 04:26
@jjw24
Copy link
Member

jjw24 commented Jul 18, 2021

i have changed target branch and doubled checked changes are the same.

@Zeroto521
Copy link
Member Author

it changed to plugin_api_v2 branch too.

@taooceros
Copy link
Member

Sorry haven't got time to double check this PR. I think it is ready to merge?

@jjw24 jjw24 merged commit 35a9d9f into Flow-Launcher:plugin_api_v2 Jul 18, 2021
@Zeroto521 Zeroto521 deleted the ci branch July 18, 2021 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants