Skip to content

Automate the merge from the List #8

Closed

Description

@avivace

This has been done manually, as a proof of concept. It should be trivially implementable.

Activity

  1. changed the title [-]Automatize the merge from the List[/-] [+]Automate the merge from the List[/+] on Feb 18, 2020
  2. ISSOtm commented on Jan 5, 2022

    @ISSOtm
    Member

    What needs to be done?

  3. avivace commented on Jan 6, 2022

    @avivace
    SponsorMemberAuthor

    What needs to be done?

    The list in this repository is based on the README.md in the awesome-gbdev, but a lot of custom elements are added, such as the video players, the "info" message about the disambiguation GB/GBA etc...

    We would need some way to template the base list and "render" it into this way, possibly with commented lines so the one in the awesome-gbdev remains clean (this pontential solution is mentioned in gbdev/awesome-gbdev#203)

  4. ISSOtm commented on Jan 6, 2022

    @ISSOtm
    Member

    The Markdown could be parsed into a list of lists (categories of entries), and that processed somehow. Replacements like "insert this between category X and Y", "process entry E thusly", etc.

  5. avivace commented on Jan 6, 2022

    @avivace
    SponsorMemberAuthor

    The Markdown could be parsed into a list of lists (categories of entries), and that processed somehow. Replacements like "insert this between category X and Y", "process entry E thusly", etc.

    It's an interesting proposal, thanks ISSO. For sure better than keeping unrelated code in the source list (even if commented).

    Considering our list is linted by awesome-lint (actually our version of it: https://github.com/gbdev/list-linter) it's already consistent to the point we could do that, but there's still a lot of spare and custom text around that would make it fail.

  6. ISSOtm commented on Jan 6, 2022

    @ISSOtm
    Member

    I'd suggest trying to go at it anyway, and seeing what hurdles come up. It may be possible to simply ignore these elements, pass them through without any processing, or give them some special treatment as with the rest, for example.

  7. avivace commented on Jan 8, 2022

    @avivace
    SponsorMemberAuthor

    I'd suggest trying to go at it anyway, and seeing what hurdles come up. It may be possible to simply ignore these elements, pass them through without any processing, or give them some special treatment as with the rest, for example.

    Ok, but it would be super low priority for me, as manually merging the list is not a big deal anyway and it happens once(maybe twice) a year. Ping me if you have time for tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions