Skip to content
This repository was archived by the owner on Aug 18, 2020. It is now read-only.

[CDEC-416] Sinbin Test.Pos.Infra.Arbitrary.Slotting #3209

Merged
merged 1 commit into from
Jul 9, 2018

Conversation

mhuesch
Copy link
Contributor

@mhuesch mhuesch commented Jul 9, 2018

Description

Free up infra-test by moving Test.Pos.Infra.Arbitrary.Slotting to sinbin-test.

Linked issue

https://iohk.myjetbrains.com/youtrack/issue/CDEC-416

Type of change

  • [~] 🐞 Bug fix (non-breaking change which fixes an issue)
  • [~] 🛠 New feature (non-breaking change which adds functionality)
  • [~] ⚠️ Breaking change (fix or feature that would cause existing functionality to change)
  • 🏭 Refactoring that does not change existing functionality but does improve things like code readability, structure etc
  • [~] 🔨 New or improved tests for existing code
  • [~] ⛑ git-flow chore (backport, hotfix, etc)

Developer checklist

  • I have read the style guide document, and my code follows the code style of this project.
  • If my code deals with exceptions, it follows the guidelines.
  • I have updated any documentation accordingly, if needed. Documentation changes can be reflected in opening a PR on cardanodocs.com, amending the inline Haddock comments, any relevant README file or one of the document listed in the docs directory.

Testing checklist

  • [~] I have added tests to cover my changes.
  • All new and existing tests passed.

@mhuesch mhuesch self-assigned this Jul 9, 2018
Since this is a module of `infra-test`, we create a new `sinbin-test`
package. Imports of the sinbinned module in `update` are modified
accordingly.
@mhuesch mhuesch force-pushed the mhuesch/CDEC-416-2 branch from 8ba1df2 to 813ba0c Compare July 9, 2018 05:17
@erikd
Copy link
Member

erikd commented Jul 9, 2018

Ignore the shellcheck failure, unless of course you'd like to fix it :)

@mhuesch
Copy link
Contributor Author

mhuesch commented Jul 9, 2018

@erikd I just tried to here but the original version (on develop) actually doesn't even run for me. It just creates copies of all the cabal files, but with -e as a suffix 🤷‍♂️

Are you able to run the script from develop successfully on your machine? If so you could test if my modified script works 🙂

@erikd
Copy link
Member

erikd commented Jul 9, 2018

The issue you're having with that script is almost certainly due to Mac's ancient BSD version of sed.

@erikd erikd merged commit 71648fa into develop Jul 9, 2018
@mhuesch mhuesch deleted the mhuesch/CDEC-416-2 branch July 9, 2018 18:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants