Skip to content

Remove eval #381

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

Merged
merged 3 commits into from
Mar 22, 2023
Merged

Remove eval #381

merged 3 commits into from
Mar 22, 2023

Conversation

GavinMendelGleason
Copy link
Member

@codecov
Copy link

codecov bot commented Feb 2, 2023

Codecov Report

Merging #381 (f4ab686) into main (bca8856) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #381      +/-   ##
==========================================
+ Coverage   71.68%   71.73%   +0.04%     
==========================================
  Files          43       43              
  Lines        4048     4048              
==========================================
+ Hits         2902     2904       +2     
+ Misses       1146     1144       -2     
Impacted Files Coverage Δ
terminusdb_client/schema/schema.py 78.80% <100.00%> (+0.17%) ⬆️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@rrooij rrooij merged commit 59a8862 into main Mar 22, 2023
@rrooij rrooij deleted the remove_eval branch March 22, 2023 10:34
GavinMendelGleason added a commit that referenced this pull request Apr 20, 2023
* Remove eval

* Remove trailing #
GavinMendelGleason added a commit that referenced this pull request Apr 20, 2023
* Add remote auth start

* Fix linting

* Use remote_auth dict for Authorization

* Fix set_db

* Fix incorrect authentication when using tokens

* Don't check db_connection on clone and fix ref

* Repeating load of object with key causes failure. (#389)

* Fix repeated key change behaviour

* Defeat the linter

* Upgrade actions to newer versions (#390)

They use an older Node version which gives a lot of deprecation
warnings in the CI.

* Fix typo (#391)

Fixes #365

* Remove eval (#381)

* Remove eval

* Remove trailing #

* Add new release for 10.2.2 (#393)

* Fix syntax of requirement

* Fix ref

---------

Co-authored-by: Robin de Rooij <[email protected]>
Co-authored-by: Robin de Rooij <[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.

2 participants