← Back to issue list

storeapi: improve to channel map docstrings

View original Github issue

Metadata

Project
snapcraft
Number
#3272
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

PR #3272 by sergiusens was merged (commit 304f28293, approved with 1 reviewer), reformatting storeapi channel map docstrings to follow PEP 257 style. Documentation-only change, +7/-19 lines in one file.

Resolution: closed resolved

Reason: The pull request itself was merged into snapcraft's master branch as commit 304f28293 ("storeapi: improve to channel map docstrings (#3272)"), confirmed in the repo git history. It carried no unresolved review comments and required no follow-up.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Follow the guideline set for docstrings in pep-257. Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved PR #3272 by sergiusens was merged (commit 304f28293, approved with 1 reviewer), reformatting storeapi channel map docstrings to follow PEP 257 style. Documentation-only change, +7/-19 lines in one file.
stealth/space-bunny-alpha — closed resolved Merged PR #3272 (commit 304f28293) improved storeapi channel map docstrings to follow PEP 257 guidelines.
qwen/qwen3.6-35b-a3b — — Merged updates to storeapi channel map docstrings, aligning them with PEP 257 guidelines. Approved by one reviewer with no unresolved comments. The change affects one file, reducing 19 lines and adding 7.
qwen3.6-35b-a3b-mtp-q6 — — Updated storeapi channel map docstrings to comply with PEP 257 guidelines. Merged into the main branch by maintainer sergiusens.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#3004 (confidence 70%)

    storeapi: add channel-map endpoint (#3004) introduced the ChannelMap code whose docstrings this PR reformatted.

  • Related To: canonical/snapcraft#3008 (confidence 65%)

    storeapi: use the channel-map api for status (#3008) was the consumer of the ChannelMap APIs documented here.

Related issues

Issue Project State Summary Similarity
#1851 storeapi: add docstrings to _client snapcraft merged Merged PR #1851 as commit ef690dba8, adding reStructuredText docstrings to the Store API _client module.
71%