Skip to content

Conversation

@emmanuellujan
Copy link
Collaborator

New JuliaParallel template, adapted to Xranklin with the help of @tlienart :)

I just have a doubt about whether Deploy.yml will work here too, but I think it will (I removed the base url prefix).

…d warnings. The issue related to variables arbitrarily set to 'nothing' persists. You must run 'serve' twice for it to work. Is this a bug in Franklin?
Adding new commits from main repo
update from juliaparallel
@emmanuellujan
Copy link
Collaborator Author

@jpsamaroo, @sloede, @giordano

Hi!, if you agree we can try to merge this new version :) (ps: if you think it's necessary please add the manifest)

Copy link
Member

@giordano giordano left a comment

Choose a reason for hiding this comment

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

A bunch of .DS_Store files have been added. They should be removed and added to .gitignore.

And yes, I still think the environment (Project + Manifest) actually used to build the website has to be added to the repository. If I want to build the website on my computer for local testing, I have no idea what to do, I can't possibly know what environment will be used in production.

@emmanuellujan
Copy link
Collaborator Author

A bunch of .DS_Store files have been added. They should be removed and added to .gitignore.

Done. Thanks.

And yes, I still think the environment (Project + Manifest) actually used to build the website has to be added to the repository. If I want to build the website on my computer for local testing, I have no idea what to do, I can't possibly know what environment will be used in production.

Is this how they are supposed to be added? or is there a cleaner way? In last deploy we apparently do not need them:
https://github.com/tlienart/juliaparallel-xr, https://tlienart.github.io/juliaparallel-xr/

@giordano
Copy link
Member

giordano commented Jun 21, 2022

Is this how they are supposed to be added? or is there a cleaner way?

For example, yes.

In last deploy we apparently do not need them:
tlienart/juliaparallel-xr, tlienart.github.io/juliaparallel-xr

Well, I presume the CI job creates an environment on-the-fly. That's completely unreproducible though, which I'm not exactly happy about. The CI job should use the environment added to the repository, so that I (or anyone else) can also build the website locally in a reproducible way.

@emmanuellujan
Copy link
Collaborator Author

Good, I added Manifest.toml and Project.toml, and I removed Manifest.toml from the gitignore.

I add the following information in case it is needed: to run the local version of project I am using a fresh Julia installation, Version 1.7.3 (2022-05-06). Then I installed Xranklin (] add https://github.com/tlienart/Xranklin.jl#dev). Finally, I execute serve(). The website is available in http://localhost:8000/.

@emmanuellujan
Copy link
Collaborator Author

In case you think it is convenient to modify the deploy (Deploy.yml) please give it a shot :) Thanks for your time

@emmanuellujan emmanuellujan requested a review from giordano July 1, 2022 14:16
@emmanuellujan
Copy link
Collaborator Author

@vchuravy

@jpsamaroo jpsamaroo merged commit e2bb415 into JuliaParallel:main Jul 1, 2022
emmanuellujan added a commit that referenced this pull request Jul 1, 2022
Merge pull request #15 from emmanuellujan/new-template
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.

3 participants