search

postgres-plr / plr

PL/R - R Procedural Language for PostgreSQL
GNU General Public License v2.0
123 stars 27 forks source link

Tagging the project no longer releases the tag #121

Closed davecramer closed 3 years ago

davecramer commented 3 years ago

For some reason we are not generating automatic releases

rheaton commented 3 years ago

Hello @davecramer Is this related to the 8.4.3 release?

We noticed that it postgres registers the version of it as 8.4.2, rather than 8.4.3 and there are not empty upgrade .sql files. Was this intentional?

Thank you, Rachel

davecramer commented 3 years ago

@rheaton ya, I was wondering about that. There are no real changes from one to the other, If this is confusing I can bump the release and change the sql file names if that helps.

rheaton commented 3 years ago

It didn't cause us any problems, but it was surprising. :)

davecramer commented 3 years ago

Apologies. I think I'll tag a release with it fixed shortly

davecramer commented 3 years ago

So nothing has changed except the release tag at this point. I guess it makes sense to bump the tag and the version number in the control file to make sure that they are the same

davecramer commented 3 years ago

@rheaton I want to tag the current master https://github.com/postgres-plr/plr/commit/dd599f2692238b866752f8f0251aab88ef1493fd This should resolve your issue. Any chance you can test it?

rheaton commented 3 years ago

Taking a look now!

rheaton commented 3 years ago

Looks good to me!

pg=# \dx;
                                      List of installed extensions
      Name      | Version |   Schema   |                          Description
----------------+---------+------------+----------------------------------------------------------------
 plr            | 8.4.4   | public     | load R interpreter and execute R script from within a database
(4 rows)
davecramer commented 3 years ago

Thx, Tagged and will release shortly using CI