8000
Skip to content

bump version 0.6.0#610

Merged
kaplanelad merged 8 commits intomasterfrom
bump-version-0.6.0
Jun 18, 2024
Merged

bump version 0.6.0#610
kaplanelad merged 8 commits intomasterfrom
bump-version-0.6.0

Conversation

@kaplanelad
Copy link
Copy Markdown
Contributor
@kaplanelad kaplanelad commented Jun 10, 2024 •
edited
Loading

Bump version to 0.6.0.

Breaking Changes Upgrade Steps

Task Signature

Change Description

Changed task args from BTreeMap to struct.

Fix:

Replace task signature
From:

async fn run(&self, app_context: &AppContext, vars: &BTreeMap<String, String>)

To:

n run(&self, _app_context: &AppContext, _vars: &task::Vars) -> Result<()>

PR: #609

Change default port

Change Description

In order to avoid the over-used over-popular port of 3000 and avoid collisions of ports, cookies, sessions and app data, we suggest moving to 5150.

PR: #611

@kaplanelad kaplanelad merged commit d496655 into master Jun 18, 2024
@BWStearns
Copy link
Copy Markdown
Contributor
BWStearns commented Jun 19, 2024

You'll also want to update calls to vars.get in tasks to vars.cli.get like this.

    async fn run(&self, app_context: &AppContext, vars: &task::Vars) -> Result<()> {
        let refresh = vars
            .cli
            .get("refresh")
            .is_some_and(|refresh| refresh == "true");

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kaplanelad @BWStearns @jondot
0