Skip to content

Conversation

@Saransh-cpp
Copy link
Owner

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes # (issue)

Type of change

Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s.

  • New feature (non-breaking change which adds functionality)
  • Optimization (back-end change that speeds up the code)
  • Bug fix (non-breaking change which fixes an issue)

Key checklist:

  • Package installs: julia> ]dev .
  • All tests pass: julia> ]test ChaoticEncryption
  • The documentation builds: $ cd docs and then $ julia make.jl

Further checks:

  • Code is commented, particularly in hard-to-understand areas
  • Tests added that prove fix is effective or that feature works

Acknowledgements

This Pull Request template was taken from the excellent PyBaMM repository.

@codecov
Copy link

codecov bot commented Mar 1, 2022

Codecov Report

Merging #52 (239230d) into master (d4610ad) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #52   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines           70        70           
=========================================
  Hits            70        70           
Impacted Files Coverage Δ
src/substitution.jl 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7fc29db...239230d. Read the comment docs.

@Saransh-cpp Saransh-cpp merged commit 1fc6338 into master Mar 1, 2022
@Saransh-cpp Saransh-cpp deleted the forgot-project.toml branch March 1, 2022 16:05
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