Skip to content

Add functions to grab commit IDs in GitHub.hx#376

Closed
NebulaStellaNova wants to merge 2 commits intoCodenameCrew:mainfrom
NebulaStellaNova:patch-5
Closed

Add functions to grab commit IDs in GitHub.hx#376
NebulaStellaNova wants to merge 2 commits intoCodenameCrew:mainfrom
NebulaStellaNova:patch-5

Conversation

@NebulaStellaNova
Copy link
Copy Markdown
Contributor

Add getCommits (Will be used on the main menu)

Add getCommits (Will be used on the main menu)
@NeeEoo
Copy link
Copy Markdown
Contributor

NeeEoo commented Sep 1, 2024

this only returns the last 30 commits btw

@NebulaStellaNova
Copy link
Copy Markdown
Contributor Author

that's fine, it just needs to be used for the most recent commit id

@NeeEoo
Copy link
Copy Markdown
Contributor

NeeEoo commented Sep 1, 2024

https://api.github.com/repos/FNF-CNE-Devs/CodenameEngine/commits

looking at this it doesnt say the amount

@NexIsDumb
Copy link
Copy Markdown
Member

and btw i dont think that GitHubRelease is the right structure for commits, you should create a new one

Copy link
Copy Markdown
Contributor Author

@NebulaStellaNova NebulaStellaNova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added get commit count

@crowplexus crowplexus changed the title Update GitHub.hx Add functions to grab commit IDs in GitHub.hx Jun 30, 2025
@crowplexus crowplexus added the type:enhancement New feature or request label Jun 30, 2025
@MRK2009
Copy link
Copy Markdown
Contributor

MRK2009 commented Apr 2, 2026

This pr is so old omg.

@r6915ee
Copy link
Copy Markdown
Contributor

r6915ee commented Apr 2, 2026

I find it pretty interesting how this has no conflicts for so long.

@NebulaStellaNova NebulaStellaNova closed this by deleting the head repository Apr 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants