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

Lines2ss #27

Closed
wants to merge 8 commits into from
Closed

Lines2ss #27

wants to merge 8 commits into from

Conversation

Yuksel-Rudy
Copy link
Contributor

I have removed the word 'test' from the example that runs line2ss function so that pytest runs without an error.

Type of change

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (non-backwards-compatible fix or feature)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Documentation update
  • Maintenance update
  • Other (please describe)

Checklist

  • I have run existing tests which pass locally with my changes
  • I have added new tests or examples that prove my fix is effective or that my feature works
  • I have added necessary documentation

@mattEhall
Copy link
Collaborator

Thanks Rudy, but when I look at the Files Changed tab where it shows the changes, it is showing various changes but not a deletion of the old file. There are also a lot of commits for just renaming a file. Could you double check your commits?

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.

2 participants