Skip to content

Prep repo for next release #54

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 5 commits into from
Jul 19, 2023
Merged

Prep repo for next release #54

merged 5 commits into from
Jul 19, 2023

Conversation

JordanMartinez
Copy link
Contributor

Description of the change

Final changes before making a new breaking release.

  • I noticed the license file was a bit outdated, especially now that node-streams-aff is integrated into this repo. I've added James explicitly to the list of people there and people in the node-purescript GH org generically. DICOM Grid, Inc. AFAIK hasn't contributed to this repo for quite some time, or at least, I'm not aware of who works for them and has contributed to this repo during company time.
  • The Main3 test wouldn't fail if the expected count differed. I've updated it to account for that now.

Checklist:

  • Added the change to the changelog's "Unreleased" section with a reference to this PR (e.g. "- Made a change (#0000)")
  • Linked any existing issues or proposals that this pull request should close
  • Updated or added relevant documentation
  • Added a test for the contribution (if applicable)

@JordanMartinez
Copy link
Contributor Author

CI is failing AFAICT because of the usage of spec in our tests due to integrating node-streams-aff into this repo. Running things locally after clearing my bower cache, I get the following interactive prompt:

Unable to find a suitable version for purescript-node-buffer, please choose one by typing one of the numbers below:
    1) purescript-node-buffer#^8.0.0 which resolved to 8.0.0 and is required by purescript-node-fs#8.2.0, purescript-node-streams#7.0.0
    2) purescript-node-buffer#^v8.0.0 which resolved to 8.0.0 and is required by purescript-spec#7.5.2
    3) purescript-node-buffer#^9.0.0 which resolved to 9.0.0 and is required by purescript-node-streams

Spec depends on node-fs and node-streams, so we have a cyclical dependency here. Even if I do override it to use node-buffer@v9.0.0, there's still breaking changes that will likely cause spec to fail to build/execute correctly.

@jamesdbrock Thoughts on how I should deal with this? While it won't be as pretty, a ReaderT Int Aff-based test setup should get us enough pretty-printing to be more legible than just using plain assert, but not by much.

@JordanMartinez
Copy link
Contributor Author

Looks like spec just declares its test deps incorrectly in its bower.json file. See the PR linked above.

@JordanMartinez JordanMartinez merged commit 1e15a3f into master Jul 19, 2023
@JordanMartinez JordanMartinez deleted the finish-repo branch July 19, 2023 18:34
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