Skip to content

Conversation

E-A-Griffin
Copy link
Contributor

Merge #98 first, fixes #100

@E-A-Griffin
Copy link
Contributor Author

@jeaye doesn't look like there's a way to test this workflow myself so probably worth running before merging in

@jeaye
Copy link
Member

jeaye commented Sep 5, 2025

@jeaye doesn't look like there's a way to test this workflow myself so probably worth running before merging in

I ran it! Failures, as expected, for now.

@jeaye
Copy link
Member

jeaye commented Sep 9, 2025

@E-A-Griffin Ok, let's update this branch and see how the CI looks.

@E-A-Griffin E-A-Griffin mentioned this pull request Sep 9, 2025
@E-A-Griffin
Copy link
Contributor Author

@jeaye tests fixed and workflow seems to be working!

@jeaye
Copy link
Member

jeaye commented Sep 9, 2025

@jeaye tests fixed and workflow seems to be working!

Huzzah! I see the deps.edn file was deleted. Is nobody using that? Or is GH just being wonky with the diff?

@E-A-Griffin
Copy link
Contributor Author

@jeaye yeah as far as I know it's unused, I can add it back though to be safe

@jeaye jeaye merged commit f1c729d into jank-lang:main Sep 10, 2025
2 checks passed
@jeaye
Copy link
Member

jeaye commented Sep 10, 2025

Excellent work, Emma!

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.

Add ClojureCLR Github Actions Workflow
2 participants