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

Error in scanner.py for python-3.8 (JWOC) #115

Open
Vinamrata1086 opened this issue Jan 23, 2021 · 6 comments
Open

Error in scanner.py for python-3.8 (JWOC) #115

Vinamrata1086 opened this issue Jan 23, 2021 · 6 comments
Assignees

Comments

@Vinamrata1086
Copy link

scanner.py

image
The low and high range is not mentioned which is low=1 and high=8888

@welcome
Copy link

welcome bot commented Jan 23, 2021

Thanks for opening your first issue!
Mentors will get back to you soon.
Meanwhile check this repo for more.

Hacktoberfest2020

@vinitshahdeo
Copy link
Owner

@Kashish121 Can you please verify the above and work with @Vinamrata1086 for the changes, if required.

@Kashish121
Copy link
Collaborator

@Vinamrata1086 That's probably due to change in the directory structure of the code. Try moving config.json to the same folder as scanner.py.

@Vinamrata1086
Copy link
Author

@Vinamrata1086 That's probably due to change in the directory structure of the code. Try moving config.json to the same folder as scanner.py.

ya it worked

so can I change the directory of config and create pr?

@Kashish121
Copy link
Collaborator

Hey @Vinamrata1086 , this is the standard organized directory structure followed in the project. Instead of changing the directory structure, try adding config.json actual path to the code. Wouldn't it be smarter? ;)

You can refer to feature\python2 or master branch for the same.

You may create a PR for this. 👍🏽

@Vinamrata1086
Copy link
Author

I have added the path.

Kashish121 added a commit that referenced this issue Feb 2, 2021
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

No branches or pull requests

3 participants