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

Python3 upgrade and a new feature to eliminate three types of python comments (1. # 2. """comment""", and 3. '''comment''' ) #44

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

maldil
Copy link

@maldil maldil commented Jul 23, 2020

I upgraded the python version of all the python files. I ran the test cases in tokenizer-unit-test.py. All test cases are working. I worked on file level tokenizer ('zip') and controller.py after the upgrade.
Changes

  1. changes for print statement
  2. Python3 zip() returns an iterator object. Since we want to iterate more than once, zip object is converted to a list to.
  3. Removed unused import statements.
  4. Python supports three ways of adding comments. 1. """ """ 2. ''' ''' 3. # . Therefore, I added another feature to specify "second_comment_open_tag".

Screen Shot 2020-07-22 at 5 06 22 PM

Feel free to contact me if you need any further changes. Thanks.

@maldil maldil changed the title Pythonupgrade and a feature to element three types of python comments (1. # 2. """comment""", and 3. '''comment''' ) Python3 upgrade and a feature to eliminate three types of python comments (1. # 2. """comment""", and 3. '''comment''' ) Jul 23, 2020
@maldil maldil changed the title Python3 upgrade and a feature to eliminate three types of python comments (1. # 2. """comment""", and 3. '''comment''' ) Python3 upgrade and a new feature to eliminate three types of python comments (1. # 2. """comment""", and 3. '''comment''' ) Jul 23, 2020
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.

1 participant