Skip to content
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

Add missing export-ignore #437

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

connorhu
Copy link
Contributor

No description provided.

@greg0ire
Copy link
Collaborator

While you're at it, can you take inspiration for other Doctrine projects regarding what to add in here? I'm thinking psalm baselines, etc.

@connorhu
Copy link
Contributor Author

@greg0ire I used to look, mostly at ORM, to see what was done and how it was done. This just came up because I did a test composer install and came down with it in the .github folder. Now that you asked, I checked and the wrong version (1.4) was install by composer, so I didn't see everything. There are more problems on dev-master. I'll update it.

@connorhu connorhu force-pushed the fix/missing-export-ignore-github branch from c811304 to 1157b82 Compare February 28, 2024 17:50
@connorhu connorhu changed the title Add missing export-ignore at .github folder Add missing export-ignore Feb 28, 2024
@greg0ire greg0ire merged commit 711eee8 into beberlei:master Feb 28, 2024
7 checks passed
@greg0ire
Copy link
Collaborator

Thanks @connorhu !

@connorhu
Copy link
Contributor Author

In the meantime I found that git archive uses it so I can test it in a local environment. One is always learning.
Thank you the suggestion!

@connorhu connorhu deleted the fix/missing-export-ignore-github branch February 28, 2024 20:30
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.

3 participants