Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@josephdangerstewart
Copy link
Contributor

No description provided.

.editorconfig Outdated Show resolved Hide resolved
.editorconfig Outdated Show resolved Hide resolved
src/Facility.CodeGen.JavaScript/JavaScriptGenerator.cs Outdated Show resolved Hide resolved
src/fsdgenjs/FsdGenJavaScriptApp.cs Outdated Show resolved Hide resolved
Co-authored-by: Ed Ball <ejball@gmail.com>
@josephdangerstewart
Copy link
Contributor Author

@ejball it looks like I don't have permission to merge this (it says it's waiting on an approval from a maintainer?), would you mind merging this for me?

image

@ejball
Copy link
Contributor

ejball commented Aug 21, 2025

@josephdangerstewart I added you to Developers.

@josephdangerstewart
Copy link
Contributor Author

Hmm, I still don't see a merge button, should I?

@ejball
Copy link
Contributor

ejball commented Aug 21, 2025

@josephdangerstewart There's probably an invitation you need to accept.

@josephdangerstewart
Copy link
Contributor Author

Ah yes, that was the issue, thanks!

@josephdangerstewart josephdangerstewart merged commit 5d2083b into FacilityApi:master Aug 21, 2025
3 checks passed
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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.