setuptools icon indicating copy to clipboard operation
setuptools copied to clipboard

[Testing] Provide a CI check for disallowed code changes

Open mohitsaxenaknoldus opened this issue 2 years ago • 7 comments

Summary of changes

Closes #3428

Pull Request Checklist

mohitsaxenaknoldus avatar Sep 04 '22 23:09 mohitsaxenaknoldus

@mohitsaxenaknoldus FYI the PR needs improvements before it's acceptable.

webknjaz avatar Oct 20 '22 14:10 webknjaz

@mohitsaxenaknoldus FYI the PR needs improvements before it's acceptable.

Sorry for the delay, I've done the requested changes. Please take a look now,

mohitsaxenaknoldus avatar Dec 03 '22 15:12 mohitsaxenaknoldus

@webknjaz Please check again.

mohitsaxenaknoldus avatar Dec 27 '22 18:12 mohitsaxenaknoldus

I saw you deleted the name, how about adding an accurate one?

webknjaz avatar Dec 27 '22 20:12 webknjaz

I saw you deleted the name, how about adding an accurate one?

Added better names.

mohitsaxenaknoldus avatar Dec 27 '22 20:12 mohitsaxenaknoldus

@jaraco @webknjaz Is this good to merge?

mohitsaxenaknoldus avatar Dec 28 '22 20:12 mohitsaxenaknoldus

@mohitsaxenaknoldus This currently has conflicts

Avasam avatar Aug 27 '24 16:08 Avasam