Skip to content

Conversation

@dacharyc
Copy link
Collaborator

@dacharyc dacharyc commented Jan 6, 2026

We have some internal tooling that outputs metrics with a specific format, similar to:

Page,Page Subsite,Measure Names,Measure Values,Min. Aux
www.example.com/docs/get-started/create-project/,Docs,Pageviews,14,1
www.example.com/docs/get-started/install/,Docs,FT Within 1 Day,2145,1

This tool can take the csv input and provide an output csv that:

  • Filters out irrelevant rows (rows that aren't related to Page views)
  • Capture the rows based on the input range, or 1-250 - i.e. "Top 250" if no input range is provided
  • Outputs just the rank and URL in ascending order for use in other contexts

This lets us get "top 50" or "top 250", or later, when we're ready, "250-300" or whatever that looks like.

I'll use the output with the Audit CLI to extract additional info to help us scope work related to these pages.

The tool creates an output csv with no headers like:

10,www.example.com/docs/get-started/install/
14,www.example.com/docs/get-started/create-project/

Copy link
Collaborator

@krollins-mdb krollins-mdb left a comment

Choose a reason for hiding this comment

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

LGTM! Just a minor comment.

}
}

func run() error {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does it make sense to print a success message when a run completes without an error?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I had it omit a success message since I thought we might use it in a pipeline - i.e. create this output and then use the output as the input in a next step. But if you'd prefer a success message, it's easy to add one!

Copy link
Collaborator

Choose a reason for hiding this comment

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

Nah, I wasn't totally sure if this was going to end up in a pipeline, but thought it might. A success message may be confusing in that case, so let's keep it out. Thanks for entertaining the thought.

Copy link
Collaborator

@krollins-mdb krollins-mdb left a comment

Choose a reason for hiding this comment

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

Nice improvements with config file. I love config files. 😆

@dacharyc dacharyc merged commit 58b5a85 into main Jan 7, 2026
1 check passed
@dacharyc dacharyc deleted the create-url-list branch January 7, 2026 23:24
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