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

Didn't catch on v135.1 and v135.2 #40

Closed
TobTobXX opened this issue Apr 5, 2022 · 5 comments
Closed

Didn't catch on v135.1 and v135.2 #40

TobTobXX opened this issue Apr 5, 2022 · 5 comments

Comments

@TobTobXX
Copy link

TobTobXX commented Apr 5, 2022

somehow the data checker didn't catch on to these tags?

@muelli
Copy link
Collaborator

muelli commented Apr 5, 2022

maybe because of the dot.
here's the regex:

@TobTobXX
Copy link
Author

TobTobXX commented Apr 5, 2022

EDIT: Nope, that's actually not the regex in use. Look at the beta branch:

However, this regex should match v135.1 or v135.2 (note that within square bracktes, the . character is already escaped). Maybe I escaped it incorrectly. I'm not sure if I even tested it, but I forgot how to. Have to test it again sometime later...

@TobTobXX
Copy link
Author

TobTobXX commented Apr 5, 2022

Hmm... it seems to work?

~/p/f/com.github.Anuken.Mindustry > flatpak run org.flathub.flatpak-external-data-checker --check-outdated com.github.Anuken.Mindustry.yml                                                      0.492s  (beta) 14:04
INFO    src.manifest: Loading sources from gradle-sources.json
INFO    src.manifest: Checking 238 external data items
INFO    src.manifest: Started check [1/238] git Mindustry (from com.github.Anuken.Mindustry.yml)
INFO    src.manifest: Skipped check [2/238] file gradle-7.3-bin.zip (from com.github.Anuken.Mindustry.yml)
INFO    src.manifest: Skipped check [3/238] file asm-3.3.1.pom (from gradle-sources.json)
[...]
INFO    src.manifest: Skipped check [238/238] file annotations-13.0.jar (from gradle-sources.json)
INFO    src.lib.externaldata: Source Mindustry: got new version 135.2
INFO    src.manifest: Finished check [238/238] git Mindustry (from com.github.Anuken.Mindustry.yml)
CHANGE SOON: Mindustry
 Has a new version:
  URL:       https://github.com/Anuken/Mindustry
  Commit:    41b22ba6a64b748b9dbcb9d0b2fdccc054f0ab17
  Tag:       v135.2
  Branch:    None
  Version:   135.2
  Timestamp: None

INFO    src.main: Check finished with 0 error(s)

No idea why the PR doesn't land.

TheEvilSkeleton added a commit that referenced this issue Apr 5, 2022
@TheEvilSkeleton
Copy link
Collaborator

Turns out, flatpak-external-data-checker doesn't automatically check for updates if the branch in question isn't the default branch. I implemented e06eb47. Hopefully everything works without a problem.

@TheEvilSkeleton
Copy link
Collaborator

#41 was created.

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

No branches or pull requests

3 participants