Skip to content

Conversation

JakeStanger
Copy link

This adds corn as a new file format.

I've enabled it by default as per the current standards, but did see discussion in #621 as to possibly changing this approach.

https://cornlang.dev

@epage
Copy link
Contributor

epage commented Sep 23, 2025

Would prefer for us to stop adding to default now and we'll remove from it in the future.

@coveralls
Copy link

coveralls commented Sep 23, 2025

Pull Request Test Coverage Report for Build 17982676208

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 64.774%

Totals Coverage Status
Change from base Build 17982617274: 0.0%
Covered Lines: 947
Relevant Lines: 1462

💛 - Coveralls

@JakeStanger
Copy link
Author

Cool, I've dropped it from the default features list

@JakeStanger
Copy link
Author

Updated the commit message to comply. Looks like the main CI job failure is unrelated?

This adds `corn` as a new file format.

https://cornlang.dev
@epage
Copy link
Contributor

epage commented Sep 24, 2025

Unsure how CI got into a red state but I just fixed it and rebased your branch on top of those fixes

@epage epage merged commit 84c3ea1 into rust-cli:main Sep 24, 2025
17 checks passed
@JakeStanger JakeStanger deleted the feat/corn-format branch September 24, 2025 16:14
@JakeStanger
Copy link
Author

Cheers for merging! I've just realised I didn't touch the readme. Want me to update that in or are you happy to?

@epage
Copy link
Contributor

epage commented Sep 24, 2025

I just pushed it

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