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

Deduplicate testing workflows #878

Open
roman-khimov opened this issue Oct 14, 2024 · 0 comments · May be fixed by #883
Open

Deduplicate testing workflows #878

roman-khimov opened this issue Oct 14, 2024 · 0 comments · May be fixed by #883
Labels
feature Completely new functionality I2 Regular impact S2 Regular significance U4 Nothing urgent

Comments

@roman-khimov
Copy link
Member

Is your feature request related to a problem? Please describe.

I'm always frustrated when we have to maintain a number of different versions of essentially the same workflow. We use testcases here, in node, for S3, for REST, but they all use slightly different versions of components. Yet we duplicated quite some code to make it work and we have to update it with every testcases update (#871 brings a new dependency? OK, we have to bring it into four repositories now).

At the same time we know workflows can be shared, like these DCO and linter ones: https://github.com/nspcc-dev/.github/tree/master/.github/workflows

Describe the solution you'd like

Create some reusable testing workflow that can be configured for specific repository and maintain it here.

@roman-khimov roman-khimov added U4 Nothing urgent S2 Regular significance I2 Regular impact feature Completely new functionality labels Oct 14, 2024
evgeniiz321 pushed a commit that referenced this issue Oct 24, 2024
closes #878

Signed-off-by: Evgeniy Zayats <[email protected]>
@evgeniiz321 evgeniiz321 linked a pull request Oct 24, 2024 that will close this issue
evgeniiz321 pushed a commit that referenced this issue Oct 24, 2024
closes #878

Signed-off-by: Evgeniy Zayats <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Completely new functionality I2 Regular impact S2 Regular significance U4 Nothing urgent
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant