Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Updated configuration files, added a comment to test/AddressSpec.hs file #55

Merged
merged 31 commits into from
Aug 12, 2024

Conversation

Oleksandr-Zhabenko
Copy link
Contributor

Updated configuration files so that the Stack LTS 20, 21, 22 are supported, added some commentary about previously noticed and probably already fixed test failures

package.yaml Outdated
@@ -30,19 +30,27 @@ dependencies:
- base >= 4.7 && < 5
- aeson
- random
- Cabal
Copy link
Member

Choose a reason for hiding this comment

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

I don't think cabal should be dependency ? Similar for other packages - why are they required ?

@Oleksandr-Zhabenko
Copy link
Contributor Author

Oleksandr-Zhabenko commented Aug 7, 2024

https://github.com/Oleksandr-Zhabenko/fakedata/actions/runs/10288712085/job/28474799400
I have changed files to meet your commentaries.

Should I open a new pull request and close this one?

@psibi
Copy link
Member

psibi commented Aug 8, 2024

Should I open a new pull request and close this one?

I think this is good if you can remove the two binary files that are added:

  • dist-newstyle/cache/config
  • dist/cabal-config-flags

@Oleksandr-Zhabenko
Copy link
Contributor Author

I updated the .gitignore so that two binary files should not be taken by Git.
https://github.com/Oleksandr-Zhabenko/fakedata/actions/runs/10305900610

@psibi
Copy link
Member

psibi commented Aug 12, 2024

I updated the .gitignore so that two binary files should not be taken by Git.

Thanks, but those files also should be removed from this PR.

@Oleksandr-Zhabenko
Copy link
Contributor Author

I have removed the two binary files, too.
https://github.com/Oleksandr-Zhabenko/fakedata/actions/runs/10350906346

@psibi
Copy link
Member

psibi commented Aug 12, 2024

Thanks, I see two things pending:

@psibi psibi merged commit 0e4dabe into fakedata-haskell:master Aug 12, 2024
9 checks passed
@psibi
Copy link
Member

psibi commented Aug 12, 2024

Thank you!

@Oleksandr-Zhabenko
Copy link
Contributor Author

You are welcome!

Oleksandr-Zhabenko added a commit to Oleksandr-Zhabenko/fakedata that referenced this pull request Aug 16, 2024
Added hyphen into domain name checker in test/TextSpec.hs

Updated configuration files, added a comment to test/AddressSpec.hs file (fakedata-haskell#55)

* Update stack.yaml

* Update stack.yaml

* Changed yaml files to use string-random-0.1.4.3

* Changed yaml files to use text-2.0.2

* Changed yaml files

* Changed yaml files

* Changed yaml files

* Changed yaml files

* Changed yaml files

* Changed package.yaml file

* Changed yaml files

* Added stack-lts-2* yaml files

* Changed workflow yaml file

* Changed workflow yaml file

* Changed workflow yaml file

* Changed workflow yaml file

* Changed stack.yaml file

* Changed stack.yaml file

* Changed stack.yaml file

* Changed stack.yaml file

* Added commentary to test/AddressSpec.hs about the reason of random tests failures

* Added commentary to test/AddressSpec.hs about the reason of random tests failures

* Changed stack.yaml

* Added missed data-files in fakedata.cabal

* Accepted changes proposed by psibi

* Accepted changes proposed by psibi for stack.yaml

* Accepted changes proposed by psibi for stack.yaml

* Fixed .github/workflows/tests.yml

* Added dist/cabal-config-flags to .gitignore file

* Removed unneeded binary files

* Removed formatting and the commentary

Changed domain name checker in test/TextSpec.hs

Sanitized fake email domain
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants