Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: FiveM #222

Merged
merged 0 commits into from
Oct 1, 2024
Merged

feat: FiveM #222

merged 0 commits into from
Oct 1, 2024

Conversation

dollannn
Copy link

No description provided.

@github-actions github-actions bot added protocol This is something regarding a protocol game This is something regarding a game labels Sep 20, 2024
@dollannn dollannn marked this pull request as ready for review September 21, 2024 15:26
@dollannn
Copy link
Author

I have no idea whether to separate info and players or to keep them together but this works for now.

@cainthebest
Copy link
Member

Approving workflow to run checks

Copy link
Member

@cainthebest cainthebest left a comment

Choose a reason for hiding this comment

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

This is starting to look good, ty for the pr.

There are some things that need doing before this can pass the CI.

  • Format the code to fmt spec with the nightly formatter
  • Add the example to the lib cargo with required features

Once this passes CI i will find the time for a full review, thanks again

@cainthebest
Copy link
Member

What happened here? Pull bot?

@github-actions github-actions bot merged commit 480ff2b into gamedig:main Oct 1, 2024
21 checks passed
@cainthebest
Copy link
Member

cainthebest commented Oct 1, 2024

@dollannn it looks like you have a bot running on your fork that is force pushing, this pr looks messed up from it.

060ccf7

@dollannn
Copy link
Author

Sorry, will have a look at this sometime in the future, dont know when this becomes relevant for me again. I was struggling with rustfmt giving me very weird formatting suggestions so I just gave up 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
game This is something regarding a game protocol This is something regarding a protocol
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants