Skip to content

🇩🇰 Adding a config for Danish#43

Merged
JonPurvis merged 1 commit intopestphp:masterfrom
berteltorp:patch-1
Apr 21, 2025
Merged

🇩🇰 Adding a config for Danish#43
JonPurvis merged 1 commit intopestphp:masterfrom
berteltorp:patch-1

Conversation

@berteltorp
Copy link
Copy Markdown
Contributor

No description provided.

@JonPurvis
Copy link
Copy Markdown
Collaborator

Hey @berteltorp 👋

Thanks very much for the contribution! The checks are running now and then I'd be happy to merge it later this evening and tag a new release.

Are you able to update the readme so that it references Dutch? There's a list of supported languages towards the bottom.

Thanks!

@stanleykinkelaar
Copy link
Copy Markdown
Contributor

Issue with the Static Tests

File '/home/runner/work/profanify/profanify/vendor/thecodingmachine/phpstan-strict-rules/phpstan-strict-rules.neon' is missing or is not readable.

and instead of referencing Dutch, as said by @JonPurvis you should reference Danish in the readme 😄

@JonPurvis
Copy link
Copy Markdown
Collaborator

Hey @berteltorp

I'm just wondering if you've got time to revisit this PR?

Thanks

@JonPurvis
Copy link
Copy Markdown
Collaborator

Issue with the Static Tests

File '/home/runner/work/profanify/profanify/vendor/thecodingmachine/phpstan-strict-rules/phpstan-strict-rules.neon' is missing or is not readable.

and instead of referencing Dutch, as said by @JonPurvis you should reference Danish in the readme 😄

Whoops, yep, that's me not knowing my languages 😆

@JonPurvis
Copy link
Copy Markdown
Collaborator

I'm going to merge this and amend the docs and then tag a new release! I hope you don't mind @berteltorp 😄 Thank you for your contribution!

@JonPurvis JonPurvis merged commit 65edff5 into pestphp:master Apr 21, 2025
18 of 20 checks passed
@JonPurvis JonPurvis mentioned this pull request Apr 21, 2025
JonPurvis added a commit that referenced this pull request Apr 21, 2025
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