Skip to content

[CLI] ETL Settings Output#4873

Merged
Akkadius merged 3 commits intomasterfrom
world-cli_etl-settings
May 15, 2025
Merged

[CLI] ETL Settings Output#4873
Akkadius merged 3 commits intomasterfrom
world-cli_etl-settings

Conversation

@joligario
Copy link
Copy Markdown
Contributor

@joligario joligario commented Apr 27, 2025

Description

The output from bin/world etl:settings dumps an event id of 0. Player Event Types start at 1.

image
image

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Testing

image

Clients tested: N/A

Checklist

  • I have tested my changes
  • I have performed a self-review of my code. Ensuring variables, functions and methods are named in a human-readable way, comments are added only where naming of variables, functions and methods can't give enough context.
  • I own the changes of my code and take responsibility for the potential issues that occur

@joligario joligario marked this pull request as ready for review April 27, 2025 14:51
@Akkadius Akkadius enabled auto-merge (squash) May 15, 2025 00:57
@Akkadius Akkadius merged commit 83cd811 into master May 15, 2025
1 of 2 checks passed
@Akkadius Akkadius deleted the world-cli_etl-settings branch May 15, 2025 01:19
@Akkadius Akkadius mentioned this pull request May 15, 2025
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.

2 participants