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

list-agreements overhaul #393

Merged
merged 7 commits into from
Jun 3, 2022
Merged

list-agreements overhaul #393

merged 7 commits into from
Jun 3, 2022

Conversation

Hevlikn
Copy link
Contributor

@Hevlikn Hevlikn commented May 29, 2022

  • handling of Deity names (as mentioned in Adding Guild Names to output of list-agreements #390)

  • age of petitions

  • added status 'expired' for outstanding petitions more than 1 year old

  • race specific guild professions

  • formatting characters to reduce mid-wo
    rd wrapping

  • consolidated functions for output and name lookup

  • minimised repeated work within if...elseif... blocks

+ handling of Deity names (as mentioned in DFHack#390)

+ age of petitions

+ added status 'expired' for outstanding petitions more than 1 year old

+ race specific guild professions

+ formatting characters to reduce mid-wo
rd wrapping

- consolidated functions for output and name lookup

- minimised repeated work within if...elseif... blocks
@Hevlikn
Copy link
Contributor Author

Hevlikn commented May 29, 2022

Example output:

image

list-agreements.lua Outdated Show resolved Hide resolved
list-agreements.lua Outdated Show resolved Hide resolved
list-agreements.lua Outdated Show resolved Hide resolved
list-agreements.lua Show resolved Hide resolved
list-agreements.lua Show resolved Hide resolved
list-agreements.lua Outdated Show resolved Hide resolved
list-agreements.lua Outdated Show resolved Hide resolved
list-agreements.lua Show resolved Hide resolved
+run arguments handler for `all` and `help`
+localised variables
+culling resolved agreements by default
+general good practices implemented
changelog.txt Outdated Show resolved Hide resolved
list-agreements.lua Outdated Show resolved Hide resolved
list-agreements.lua Show resolved Hide resolved
list-agreements.lua Show resolved Hide resolved
list-agreements.lua Show resolved Hide resolved
list-agreements.lua Outdated Show resolved Hide resolved
list-agreements.lua Outdated Show resolved Hide resolved
list-agreements.lua Outdated Show resolved Hide resolved
list-agreements.lua Outdated Show resolved Hide resolved
list-agreements.lua Outdated Show resolved Hide resolved
list-agreements.lua Outdated Show resolved Hide resolved
@Hevlikn Hevlikn requested a review from myk002 June 1, 2022 04:07
@myk002 myk002 merged commit 3a59e56 into DFHack:master Jun 3, 2022
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