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

[Lang] Fix pylint rule C0200 #3480

Merged
merged 9 commits into from Nov 13, 2021
Merged

[Lang] Fix pylint rule C0200 #3480

merged 9 commits into from Nov 13, 2021

Conversation

kemingy
Copy link
Contributor

@kemingy kemingy commented Nov 12, 2021

Related issue = #3151

@CLAassistant
Copy link

CLAassistant commented Nov 12, 2021

CLA assistant check
All committers have signed the CLA.

@netlify
Copy link

netlify bot commented Nov 12, 2021

✔️ Deploy Preview for jovial-fermat-aa59dc canceled.

🔨 Explore the source changes: 070028e

🔍 Inspect the deploy log: https://app.netlify.com/sites/jovial-fermat-aa59dc/deploys/618fa8086ce78a0007f0ea69

@kemingy
Copy link
Contributor Author

kemingy commented Nov 13, 2021

Can anyone approve the CI workflow? 😄 Otherwise I have to merge master again and again to update the CI pylint check settings.

@k-ye
Copy link
Member

k-ye commented Nov 13, 2021

Can anyone approve the CI workflow? smile Otherwise I have to merge master again and again to update the CI pylint check settings.

Sure, repushed & retriggered :-)

@yihong0618
Copy link
Collaborator

LGTM

@sjwsl sjwsl self-assigned this Nov 13, 2021
@sjwsl sjwsl merged commit 702675e into taichi-dev:master Nov 13, 2021
@kemingy kemingy deleted the pylint_c0200 branch November 13, 2021 13:13
@lucywsq
Copy link
Contributor

lucywsq commented Dec 2, 2021

Hi, @kemingy We are so appreciative of your contribution to fixing pylint rules through the “Let’s become Taichi contributors within 10 minutes” program. Please send an email to siqiwu@taichi.graphics with your offline address. We would deliver a hoodie to you. : ) Looking forward to hearing from you.

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.

None yet

6 participants