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(content): Smart watches can sense wind, addition of "WINDMETER" flag. #4645

Merged
merged 3 commits into from
May 13, 2024

Conversation

Lamandus
Copy link
Contributor

@Lamandus Lamandus commented May 13, 2024

Purpose of change

There was only one Object that could sense the wind, that was the "Doppler Radar Turbo 2000". Cringy name, I know. And it dropped only by zombie scientist by a very small change. Instead of relying on that thing, we now have the flag "WINDMETER" that can print out weather information (as barometer, thermometer and gyrometer).

Already you can buy Wind meter accessories for your phone, so why shouldn't we have it build in into your smartphone. https://mrtarget.de/skywatch-windoo-2-wind-meter-windmesser-anemometer-fuer-das-smartphone (not sponsored)

Describe the solution

Add Windmeter as a flag. This is only used by the Turbo thing and the Smart Watch for now. Can be expanded to other object in the future.

Describe alternatives you've considered

Testing

spawned a smart watch and let it happen

Additional context

grafik

Checklist

@github-actions github-actions bot added src PR changes related to source code. data PRs related to datas. Won't crash game (probably) labels May 13, 2024
Copy link
Contributor

autofix-ci bot commented May 13, 2024

Autofix has formatted code style violation in this PR.

I edit commits locally (e.g: git, github desktop) and want to keep autofix
  1. Run git pull. this will merge the automated commit into your local copy of the PR branch.
  2. Continue working.
I do not want the automated commit
  1. Format your code locally, then commit it.
  2. Run git push --force to force push your branch. This will overwrite the automated commit on remote with your local one.
  3. Continue working.

If you don't do this, your following commits will be based on the old commit, and cause MERGE CONFLICT.

@Lamandus
Copy link
Contributor Author

if it gets merged, @chaosvolt can you be so kind and post about it in the json-changes discord? thx.

src/iuse.cpp Show resolved Hide resolved
@chaosvolt chaosvolt merged commit cc6e3ee into cataclysmbnteam:main May 13, 2024
13 checks passed
@Lamandus Lamandus deleted the windy branch May 15, 2024 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data PRs related to datas. Won't crash game (probably) src PR changes related to source code.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants