Skip to content

Create file_utils.py #2735

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

Closed
wants to merge 7 commits into from
Closed

Conversation

MonarchChakri
Copy link

Utility file helpful for file I/O in python3.
Helps for plain text IO and JSON load and dump.

Describe your change:

  • Add an algorithm?
  • Fix a bug or typo in an existing algorithm?
  • Documentation change?

Checklist:

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized.
  • I know that pull requests will not be merged if they fail the automated tests.
  • This PR only changes one algorithm file. To ease review, please open separate PRs for separate algorithms.
  • All new Python files are placed inside an existing directory.
  • All filenames are in all lowercase characters with no spaces or dashes.
  • All functions and variable names follow Python naming conventions.

Utility file helpful for file I/O in python3.
Helps for plain text IO and JSON load and dump.
Updated to make it compatible with black
@MonarchChakri
Copy link
Author

Hi @cclauss / @poyea
Added file_utils.py under others and created this PR for #2510
A utility files helpful for all common file IO.
Please review and merge.

@MonarchChakri
Copy link
Author

@cclauss
Will make the changes and update the file

MonarchChakri and others added 5 commits October 4, 2020 12:58
Co-authored-by: Christian Clauss <[email protected]>
Co-authored-by: Christian Clauss <[email protected]>
Correcting black and flake8 changes
black: Single space after file_name
@MonarchChakri
Copy link
Author

@cclauss
Updated the file, please review

@MonarchChakri MonarchChakri requested a review from cclauss October 4, 2020 08:34
@poyea poyea added the hacktoberfest-accepted Accepted to be counted towards Hacktoberfest label Oct 5, 2020
@MonarchChakri
Copy link
Author

@poyea
Can you also please approve and merge. 😅

@stale
Copy link

stale bot commented Nov 21, 2020

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Used to mark an issue or pull request stale. label Nov 21, 2020
@stale
Copy link

stale bot commented Nov 29, 2020

Please reopen this pull request once you commit the changes requested or make improvements on the code. If this is not the case and you need some help, feel free to seek help from our Gitter or ping one of the reviewers. Thank you for your contributions!

@stale stale bot closed this Nov 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Accepted to be counted towards Hacktoberfest stale Used to mark an issue or pull request stale.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants