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

Move Workflow List Files to New Location [GCC2024_COFEST] #18466

Merged
merged 2 commits into from
Jul 3, 2024
Merged

Move Workflow List Files to New Location [GCC2024_COFEST] #18466

merged 2 commits into from
Jul 3, 2024

Conversation

itisAliRH
Copy link
Member

This PR moves all workflow list related files from components/Workflow to components/Workflow/List.
Requires #18465

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

@itisAliRH itisAliRH requested a review from guerler June 28, 2024 15:05
@itisAliRH itisAliRH force-pushed the workflow-list-move-files branch from 705b110 to d26e504 Compare July 2, 2024 12:10
guerler
guerler approved these changes Jul 3, 2024
View reviewed changes
Copy link
Contributor

@guerler guerler left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@guerler guerler merged commit b03b21a into galaxyproject:dev Jul 3, 2024
30 checks passed
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 3, 2024

This PR was merged without a "kind/" label, please correct.

@guerler guerler added the kind/refactoring cleanup or refactoring of existing code, no functional changes label Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@guerler guerler guerler approved these changes

Assignees
No one assigned
Labels
area/UI-UX kind/refactoring cleanup or refactoring of existing code, no functional changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@itisAliRH @guerler