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

copy & paste #33

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

copy & paste #33

wants to merge 2 commits into from

Conversation

m4xr1sk
Copy link

@m4xr1sk m4xr1sk commented Jan 29, 2022

No description provided.

@tchapi
Copy link
Owner

tchapi commented Jan 29, 2022

Hi
What is the purpose of this PR ?

Thanks

@tchapi tchapi added the invalid label Jan 29, 2022
@m4xr1sk
Copy link
Author

m4xr1sk commented Jan 29, 2022 via email

@tchapi
Copy link
Owner

tchapi commented Jan 29, 2022

It seems that they are a lot of changes in your commits that are not related to that (putting scripts in different files, indentation, etc) so I cannot merge as-is.

Try to extract that specific feature (copy & paste) so the changeset is readable and contained so I can review it

thanks !

@m4xr1sk
Copy link
Author

m4xr1sk commented Jan 29, 2022 via email

@m4xr1sk
Copy link
Author

m4xr1sk commented Jan 29, 2022

I have reverted the changes then added just the few lines to do the copy and paste. Now the 2 branches are comparable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants