-
Notifications
You must be signed in to change notification settings - Fork 1
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
💅 polishing off the tasking w/ prettify #387
Comments
There is a tap reporter named prettify — https://www.npmjs.com/package/tap-prettify — also usable for this issue since the verification and formatting tasks seem like tests, and we could probably have output via tap reporter, but that wasn't what i originally meant. |
related: OpenINF/openinf-util-text#69 |
Here's the PR! #483.⚡ Sweep Free Trial: I used GPT-4 to create this ticket. You have 4 GPT-4 tickets left. For more GPT-4 tickets, visit our payment portal.
Step 1: 🔍 Code SearchI found the following snippets in your repository. I will now analyze these snippets and come up with a plan. Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description..github/.github/workflows/lint-and-test.yml Lines 1 to 92 in 071e7bf
Lines 41 to 205 in 071e7bf
Lines 1 to 148 in 071e7bf
Lines 126 to 210 in 071e7bf
.github/build/tasks/format/format-md.mjs Lines 1 to 28 in 071e7bf
Step 2: 🧐 Snippet AnalysisFrom looking through the relevant snippets, I decided to make the following modifications:
Step 3: 📝 PlanningI have created a plan for writing the pull request. I am now working my plan and coding the required changes to address this issue. Here is the planned pull request: Polishing off the tasking with prettify
Step 4: ⌨️ CodingI have finished coding the issue. I am now reviewing it for completeness. Step 5: 🔁 Code ReviewSuccess! 🚀 I'm a bot that handles simple bugs and feature requests but I might make mistakes. Please be kind! |
Our tasking situation here will get a lot more impressive and be less-incomplete once the changes i had proposed in ampproject/amp.dev#3346 finally make their way over here. That is quite a high priority since it precludes the org-wide adoption of our tasking structure here. The funny thing about our currently-incomplete tasking situation here is that it is very home-grown, and although it may have already outgrown its original training wheels (the
nps
solution), they are still fully-functional and even still used by the CI to this day.The text was updated successfully, but these errors were encountered: