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

Run button now shows paddle's Run Configuration #63

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

khbminus
Copy link
Collaborator

Closes #33

@khbminus khbminus requested a review from SmirnovOleg May 10, 2023 17:04
@khbminus khbminus self-assigned this May 10, 2023
@khbminus khbminus changed the base branch from main to configuration May 17, 2023 11:58
@khbminus khbminus changed the base branch from configuration to main May 17, 2023 11:58
Copy link
Collaborator

@SmirnovOleg SmirnovOleg left a comment

Choose a reason for hiding this comment

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

@khbminus I guess that the logic for tests is a bit improper: for example project, when I click on Run button on the gutter for any test in example/subproject/test/test_example.py file, it runs ALL the tests in the file. Is it because it finds a test task for parent in the paddle.yaml?

For other cases - looks awesome. Nice job!

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.

Overload Run __main__ button on the gutter for if __name__ == '__main__' line
2 participants