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

GitHub Actions: create a tag to our repo weekly if a new change is committed since last tagging #2655

Merged
merged 3 commits into from Oct 9, 2020

Conversation

masatake
Copy link
Member

@masatake masatake commented Oct 3, 2020

Partially close #1038.

The tagging format is v5.9.YYYYMMDD.0.
The tagging format is p5.9.YYYYMMDD.0.

@masatake masatake requested a review from k-takata October 3, 2020 21:53
@masatake masatake mentioned this pull request Oct 3, 2020
@coveralls
Copy link

coveralls commented Oct 3, 2020

Coverage Status

Coverage remained the same at 86.998% when pulling e9baeff on masatake:weekly-git-tagging into d93b1a8 on universal-ctags:master.

Copy link
Member

@k-takata k-takata left a comment

Choose a reason for hiding this comment

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

Maybe we should update src/ctags.h and configure.ac to 5.9.0?
(If they are updated, tests should be also updated, though...)

.github/workflows/tagging.yml Outdated Show resolved Hide resolved
misc/git-tag-maybe.sh Outdated Show resolved Hide resolved
@masatake
Copy link
Member Author

masatake commented Oct 6, 2020

Maybe we should update src/ctags.h and configure.ac to 5.9.0?
(If they are updated, tests should be also updated, though...)

Do you mean updating the version strings in the files WHEN tagging?

@k-takata
Copy link
Member

k-takata commented Oct 6, 2020

I think it's better to set the major and the minor numbers to the same as the tag name.
I don't want to say that the patch number should be updated on every tagging.

@masatake
Copy link
Member Author

masatake commented Oct 6, 2020

@k-takata, thank you for reviewing. I updated this pull request.
I cherry-pick'ed the @albfan's work.

I have a question.

win32/ctags.exe.manifest:  <assemblyIdentity type="win32" name="Universal Ctags" version="0.0.0.0" />
win32/ctags.rc:            VALUE "FileVersion", "0.0.0.0"
win32/ctags.rc:            VALUE "ProductVersion", "0.0.0.0"
``

Should I update these files? If the answer is yes, how? "5.9.0.0" is o.k.?

@k-takata
Copy link
Member

k-takata commented Oct 6, 2020

Should I update these files? If the answer is yes, how? "5.9.0.0" is o.k.?

Yes, "5.9.0.0" should be okay.

@masatake masatake mentioned this pull request Oct 6, 2020
@masatake
Copy link
Member Author

masatake commented Oct 6, 2020

@k-takata,
Could you see b34696f, the change for win32?

@k-takata
Copy link
Member

k-takata commented Oct 7, 2020

Could you see b34696f, the change for win32?

Looks okay.

Comment on lines 10 to 12
desc=$(git describe --tags --always)
case "${desc}" in
v*)
Copy link
Member

Choose a reason for hiding this comment

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

This may not work as expected.
If once v5.9.20201007.0 was tagged and some commits were added, git describe --tags --always would return v5.9.20201007.0-N-gHHHHHHH. (N is a number of commits from the last tag, HHHHHHH is the commit ID.)
If --exact-match option is specified, git describe will fail if the current commit is not tagged.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thank you. I tried --exact-match but it seems to me that the optoin doesn't work expectedly:

$ git describe --tags --always
9ce35430
$ git tag x
$ git describe --exact-match  
fatal: no tag exactly matches '9ce35430b0866d055fb18090d85bd00c1dea43d0'
$ echo $?
128
$ git describe --tags --always
x

How about pattern

   v*.0)

instead of v* ?
Too stingy?

Copy link
Member

Choose a reason for hiding this comment

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

How about git describe --tags --exact-match?

@masatake masatake changed the title GitHub action: create a tag to our repo weekly if a new change is committed since last tagging GitHub actions: create a tag to our repo weekly if a new change is committed since last tagging Oct 8, 2020
@masatake
Copy link
Member Author

masatake commented Oct 8, 2020

I decide to use p instead of v. I would like to reserve v for the future.
p means Peridoical.

@masatake masatake changed the title GitHub actions: create a tag to our repo weekly if a new change is committed since last tagging GitHub Actions: create a tag to our repo weekly if a new change is committed since last tagging Oct 8, 2020
@masatake
Copy link
Member Author

masatake commented Oct 8, 2020

The change of prefix must be written in a new blog entry.

@codecov
Copy link

codecov bot commented Oct 8, 2020

Codecov Report

Merging #2655 into master will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2655   +/-   ##
=======================================
  Coverage   86.90%   86.90%           
=======================================
  Files         183      183           
  Lines       39016    39018    +2     
=======================================
+ Hits        33906    33908    +2     
  Misses       5110     5110           
Impacted Files Coverage Δ
main/parse.c 95.09% <0.00%> (-0.01%) ⬇️
main/ptag.c 93.58% <0.00%> (+0.25%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e852ee0...e9baeff. Read the comment docs.

albfan and others added 3 commits October 10, 2020 05:37
Updated to 5.9.0.

@masatake updated some more files.
@masatake edited this commit log.
…mmitted since last tagging

Partially close universal-ctags#1038.

The tagging format is p5.9.YYYYMMDD.0.

@k-takata helped me utilize git describe/tag.
@masatake
Copy link
Member Author

masatake commented Oct 9, 2020

Simplified the commits.

@masatake
Copy link
Member Author

masatake commented Oct 9, 2020

Let's see what happens in this Sunday.

@masatake masatake merged commit a111a61 into universal-ctags:master Oct 9, 2020
@masatake
Copy link
Member Author

masatake commented Oct 9, 2020

5.9.0 are hard-coded in many places. I must reduce them.

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.

Release plans ?
4 participants