Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

removed global #48

Merged
merged 2 commits into from
Oct 7, 2021
Merged

removed global #48

merged 2 commits into from
Oct 7, 2021

Conversation

rupesh-dharme
Copy link
Contributor

I have removed the use of global keyword and replaced it with two functions.

@ghost
Copy link

ghost commented Oct 7, 2021

Changes to your CodeSee Architecture Map:

View a CodeSee Map of these changes

View more CodeSee Maps!

Legend

CodeSee Map Legend

@ZigRazor ZigRazor added enhancement New feature or request hacktoberfest hacktoberfest issue performance Performance enhancement labels Oct 7, 2021
@ZigRazor ZigRazor self-assigned this Oct 7, 2021
@ZigRazor ZigRazor linked an issue Oct 7, 2021 that may be closed by this pull request
@rupesh-dharme
Copy link
Contributor Author

Why does it say merging is blocked? Do I need to do something?

@ZigRazor
Copy link
Owner

ZigRazor commented Oct 7, 2021

Why does it say merging is blocked? Do I need to do something?

it's ok just waiting the CI ends

@ZigRazor ZigRazor merged commit 4ae032d into ZigRazor:main Oct 7, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request hacktoberfest hacktoberfest issue performance Performance enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

(PYL-W0603) global statement detected
2 participants