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

Pr merged test #1

Closed
wants to merge 2 commits into from
Closed

Conversation

MrZilinXiao
Copy link
Owner

No description provided.

@MrZilinXiao MrZilinXiao deleted the pr_merged_test branch August 30, 2022 02:35
MrZilinXiao added a commit that referenced this pull request Sep 6, 2022
(Need discussion) and (TODO) flags added for further discussion
MrZilinXiao added a commit that referenced this pull request Oct 7, 2022
* inital changes

* changes

* changes

* changes

* changes

* changes

* fix formatting for test_modules

* changes

* fix some formatting errors for our new tests

* added docstrings for most but not all methods

* add basic docstring outlines for all functions in module.py, need descriptions for functions, arguments and retursn

* docstring add #1
(Need discussion) and (TODO) flags added for further discussion

* add docstring ivy-llc#2

* add separation of positional-only arguments and keyword-only arguments

* fix syntax error of module.py

* changes and fixes

* fixes

* lint fixes

* changes

* changes

* changes

* changes;

* conftest update deadline

* changes made as requested

* changes:

Co-authored-by: Rishabh Kumar <[email protected]>
Co-authored-by: Daniel4078 <[email protected]>
Co-authored-by: Zilin Xiao <[email protected]>
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.

1 participant