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

Adding 3 new functions: Integers, PrependTo and ContainsOnly #691

Merged
merged 3 commits into from
Jun 5, 2019

Conversation

larionvn
Copy link
Contributor

This pull request includes the implementation of the following functions:

  • Integers[]
  • PrependTo[]
  • ContainsOnly[]

Note: The IPython API version 6.0+ does not support Python 2.6, 2.7, 3.0, 3.1, or 3.2. So I have changed the .travis.yml to install IPython 5.0 for covering all the test cases.

@larionvn
Copy link
Contributor Author

Hi @sn6uv, can you help me to restart build for this pull request. The build has been failed with the error code (connection reset by peer) so it should be a server issue, not from the code. I've also passed all checks several times in my local repos . Thanks you in advance.

@sn6uv
Copy link
Member

sn6uv commented May 12, 2017

Just restarted it now. Sorry about the delay.

Copy link
Member

@sn6uv sn6uv left a comment

Choose a reason for hiding this comment

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

Just one thing to fix up

mathics/builtin/lists.py Outdated Show resolved Hide resolved
@wolfv wolfv force-pushed the new_functions1 branch from 3581622 to 9741e8c Compare June 5, 2019 15:31
@wolfv
Copy link
Member

wolfv commented Jun 5, 2019

this one seems good to go, too :)

@wolfv wolfv merged commit 84689fa into mathics:master Jun 5, 2019
@rocky
Copy link
Member

rocky commented Sep 16, 2020

@larionvn and @GarkGarcia I did the rebase - please double check though. I make a lot of mistakes. If it is addressed, it would be good to close this.

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.

4 participants