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

Hacktober fest raje pr1 #226

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

Conversation

rajeshsgr
Copy link

Code Snippet for File Rename

/*
* MIT License
*
* Copyright (c) 2017-2022 Ilkka Seppälä

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Copyright (c) 2017-2024 Ilkka Seppälä

@Ritabrata1080
Copy link

Changes LGTM ✅.

Shall the copyright year be 2017-2024 or 2017-2022?
Please confirm @iluwatar

Copy link

@Ritabrata1080 Ritabrata1080 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please squash all the commits to a single commit.

@rajeshsgr
Copy link
Author

I have done all the changes, the build is failing for a file which i have not changed or worked on

@Ritabrata1080
Copy link

I have done all the changes, the build is failing for a file which i have not changed or worked on

Approved. Please squash the commits once.

File Rename Snippet

Code Snippet to Rename a File

Test class for Code Snippet to Rename a File
@rajeshsgr
Copy link
Author

Squashed, please check

@Ritabrata1080
Copy link

This checks might be failing due to addition of main() in the code. I think only driver code is allowed to be pushed and not the main method.

@rajeshsgr
Copy link
Author

I have removed the main, it fails when it compiles - MultiArrayConcatenationSnippet.java and says it uses unchecked or unsafe operations. This is a file which i have not changed or worked on .

@Ritabrata1080
Copy link

I have removed the main, it fails when it compiles - MultiArrayConcatenationSnippet.java and says it uses unchecked or unsafe operations. This is a file which i have not changed or worked on .

I also faced many issues to get the scans successful. Not sure why MultiArrayConcatenationSnippet.java file compiled for me successfully. Is there any channel where we collaborators can connect?

@Ritabrata1080
Copy link

Also, could u plz review my PR #224?

Copy link
Owner

@iluwatar iluwatar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check the following

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.

3 participants