Skip to content

Update dependency to remove warning#255

Open
osenan wants to merge 5 commits into
devfrom
update-dependency-warning@dev
Open

Update dependency to remove warning#255
osenan wants to merge 5 commits into
devfrom
update-dependency-warning@dev

Conversation

@osenan

@osenan osenan commented Dec 12, 2025

Copy link
Copy Markdown

It solves #238
Was caused by problem in teal.modules.general.
This is the companion PR.

To test it, check the branch, restore dependencies and open the app. Unlike in dev branch, you should not see any warning.

@osenan osenan added the core label Dec 12, 2025
@github-actions

github-actions Bot commented Dec 12, 2025

Copy link
Copy Markdown
Contributor


🎉 Thank you for your contribution! Before this PR can be accepted, we require that you all read and agree to our Contributor License Agreement.
You can digitally sign the CLA by posting a comment on this Pull Request in the format shown below. This agreement will apply to this PR as well as all future contributions on this repository.


I have read the CLA Document and I hereby sign the CLA


1 out of 2 committers have signed the CLA.
✅ (averissimo)[https://github.com/averissimo]
@osenan
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

Signed-off-by: Oriol Senan  <35930244+osenan@users.noreply.github.com>
@osenan osenan requested a review from m7pr December 12, 2025 14:05
@llrs-roche

Copy link
Copy Markdown
Contributor

This might be duplicate effort from #250. I would wait to see if that one fixes the deployment or at least compare between the branches

Comment thread RNA-seq/renv.lock
"Author": "Stefan Milton Bache [aut, cph] (Original author and creator of magrittr), Hadley Wickham [aut], Lionel Henry [cre], Posit Software, PBC [cph, fnd] (ROR: <https://ror.org/03wc8by49>)",
"Maintainer": "Lionel Henry <lionel@posit.co>",
"Repository": "CRAN"
"Repository": "https://packagemanager.posit.co/cran/__linux__/jammy/latest"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should remain as CRAN or RSPM

Comment thread RNA-seq/renv.lock
"Author": "Gábor Csárdi [aut, cre]",
"Maintainer": "Gábor Csárdi <csardi.gabor@gmail.com>",
"Repository": "CRAN"
"Repository": "https://packagemanager.posit.co/cran/__linux__/jammy/2025-10-29"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should not be explicit, keep CRAN or use RSPM

Comment thread RNA-seq/renv.lock
"RemoteRepo": "teal.modules.general",
"RemoteRef": "main",
"RemoteSha": "8d2e48b6f7a5764f60972c69a714395cad32a516"
"RemoteSha": "6604ddefaae0f85577fa6065d65b4bd23f41afa6"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If I understand the workflows this will be automatically updated by the bot daily, there is no need for a PR.

If the change is urgent, then please make the PR a bit more minimal as the bot does

See this example commit:

8e546ac

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants