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

Create True/False generator #332

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

Conversation

AaryaNale
Copy link

This Python script provides a simple way to generate all possible combinations of True and False values for a given list of variables.
Is widely used in inputs for problems in electronics.

@AaryaNale
Copy link
Author

Please review and merge. Thank you.

@hastagAB
Copy link
Owner

hastagAB commented Oct 4, 2023

Hi @AaryaNale, please follow our Contribution Guidelines

@AaryaNale
Copy link
Author

@hastagAB Can you please let me know about the modifications i need to make?
Thank you.

@AaryaNale
Copy link
Author

@hastagAB
I reviewed the Contribution Guidelines. The only two things absent in my file are:

  • Requiremnts.txt file
  • Haven't added my name to the project list.
    At this moment, I cannot inculcate both, because:
  • My code does not require specific external dependencies to run.
  • I cannot add my name to the project list because it has not been merged into the master branch yet, and i do not have a project URL to redirect to.

If there is something I am missing out, please let me know, and I'll try to improve ASAP.

@hastagAB
Copy link
Owner

hastagAB commented Oct 6, 2023

@AaryaNale Requirements.txt file is optional and in case you don't have any external dependency you can skip this.
You can add your name in the main README keeping the links relative to your forked repo. Merging your PR will add your script, as well as your README line contribution. Both will be reflected in the main repo once merged.
Thanks!

@AaryaNale AaryaNale mentioned this pull request Oct 6, 2023
@AaryaNale
Copy link
Author

AaryaNale commented Oct 6, 2023

@hastagAB
I have added the entry and mentioned this PR. Please let me know if any other changes are to be made.
For reference, the README.md PR is Update README.md #339
Thanks.

@hastagAB
Copy link
Owner

hastagAB commented Oct 9, 2023

Hi @AaryaNale, you don't need to open a separate PR for the README updation. Just make the changes in your existing codebase the push the changes, and it'll automatically reflect in this PR.

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.

2 participants