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

A couple of internal changes, improvements, and fixes #430

Merged
merged 15 commits into from
Jan 14, 2023
Merged

Conversation

AmyrAhmady
Copy link
Collaborator

@AmyrAhmady AmyrAhmady commented Jun 16, 2022

  • Remove most of boost containers and use std ones
  • Set C++ std version to 17
  • Remove previous boost submodule and replace it with a header only one
  • Fix IsDynamicActorStreamedIn params order
  • Fix actor default virtual world, it is now set to 0
  • Ue github actions for ci, and remove appveyor/travis
  • Proper gitignore additions

src/utility/misc.h Outdated Show resolved Hide resolved
@samp-incognito
Copy link
Owner

Could you also replace the build status badges in README.md with this one?

[![Build Status](https://github.com/samp-incognito/samp-streamer-plugin/actions/workflows/build.yml/badge.svg)](https://github.com/samp-incognito/samp-streamer-plugin/actions/workflows/build.yml)

Build Status

@Fairuz-Afdhal
Copy link

Can you also add:
Streamer_HasFloatData and Streamer_RemoveFloatData.

Also there's an issue with E_STREAMER_CUSTOM as it accepts integer data related only instead of them all. (Streamer_GetIntData, Streamer_SetIntData). It just would print Invalid data specified.

@AmyrAhmady
Copy link
Collaborator Author

Can you also add: Streamer_HasFloatData and Streamer_RemoveFloatData.

Also there's an issue with E_STREAMER_CUSTOM as it accepts integer data related only instead of them all. (Streamer_GetIntData, Streamer_SetIntData). It just would print Invalid data specified.

Please create an issue for this so I don't forget

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