Skip to content

Commit

Permalink
Switch from REQUIRE to Project.toml
Browse files Browse the repository at this point in the history
  • Loading branch information
Bernard Godard committed May 27, 2019
1 parent 1a4ea22 commit 6e61582
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 3 deletions.
19 changes: 19 additions & 0 deletions Project.toml
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
name = "CALCEPH"
uuid = "1537fe66-4725-5aba-80f4-3a74792cecc1"
authors = ["Bernard Godard"]
version = "1.0.2"

[deps]
BinDeps = "9e28174c-4ba2-5203-b857-d8d62c4213ee"
BinaryProvider = "b99e7846-7c00-51b0-8f62-c81ae34c0232"

[compat]
BinDeps = "≥ 0.8.1"
BinaryProvider = "≥ 0.1.4"
julia = "≥ 0.7.0"

[extras]
Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"

[targets]
test = ["Test"]
3 changes: 0 additions & 3 deletions REQUIRE
View file

This file was deleted.

10 comments on commit 6e61582

@bgodard
Copy link
Member

@bgodard bgodard commented on 6e61582 May 27, 2019 •
edited by helgee
Loading

Choose a reason for hiding this comment

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

@JuliaRegistrator register

@helgee
Copy link
Member

@helgee helgee commented on 6e61582 May 27, 2019 •
edited
Loading

Choose a reason for hiding this comment

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

@JuliaRegistrator register()

@helgee
Copy link
Member

@helgee helgee commented on 6e61582 May 27, 2019

Choose a reason for hiding this comment

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

@JuliaRegistrator register()

@JuliaRegistrator
Copy link

Choose a reason for hiding this comment

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

Error while trying to register: Changing package repo URL not allowed, please submit a pull request with the URL change to the target registry and retry.

@helgee
Copy link
Member

@helgee helgee commented on 6e61582 May 27, 2019

Choose a reason for hiding this comment

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

Okay, @bgodard. It seems to be picky about the syntax. This works:

@JuliaRegistrator `register()`

But now we got the problem above...

@JuliaRegistrator
Copy link

Choose a reason for hiding this comment

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

Error while trying to register: Changing package repo URL not allowed, please submit a pull request with the URL change to the target registry and retry.

@giordano
Copy link
Member

@giordano giordano commented on 6e61582 May 27, 2019 •
edited
Loading

Choose a reason for hiding this comment

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

I think the syntax is simply

@JuliaRegistrator register()

without any quote. Also, editing the comment doesn't work, the bot doesn't listen to edits

@JuliaRegistrator
Copy link

Choose a reason for hiding this comment

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

Error while trying to register: Changing package repo URL not allowed, please submit a pull request with the URL change to the target registry and retry.

@giordano
Copy link
Member

@giordano giordano commented on 6e61582 May 27, 2019 •
edited
Loading

Choose a reason for hiding this comment

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

The problem it's reporting is that here https://github.com/JuliaRegistries/General/blob/2260a6551b0023329cc54fd83eba7ed343bbcf37/C/CALCEPH/Package.toml there is still the old URL, you should submit a pull request to update it

@bgodard
Copy link
Member

@bgodard bgodard commented on 6e61582 May 27, 2019 •
edited
Loading

Choose a reason for hiding this comment

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

Thank you @helge and @giordano for investigating.

I have made the pull request on JuliaRegistries.

According to this page https://github.com/JuliaRegistries/Registrator.jl the call to register can be made without a (). I will open an issue there.

Please sign in to comment.