Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor markdown link fix in orchestration docs #8660

Merged
merged 1 commit into from Feb 27, 2023
Merged

Minor markdown link fix in orchestration docs #8660

merged 1 commit into from Feb 27, 2023

Conversation

mjschock
Copy link
Contributor

Example

Updates the docs here from:

Instantiate [an asynchronous PrefectClient][/api-ref/prefect/client/orchestration/#prefect.client.orchestration.PrefectClient] within Python to send requests to the API.

to:

Instantiate an asynchronous PrefectClient within Python to send requests to the API.

Incidentally, I ran pytest tests, noticed a few artifacts were generated, and tracked that to tests/cli/deployment/test_deployment_build.py, which creates dog_flow_func-deployment.yaml, dog-deployment.yaml, and f.py. It looks like .prefectignore is also automatically being generated when running the CLI deployment tests.

I also noticed another error in tests/cli/deployment/test_deployment_run.py:

AssertionError: Desired contents 'Scheduled start time: 2023-03-13 11:47:14' not found in CLI ouput "Creating flow run for deployment 'my-flow/My Deployment'...\nCreated flow run 'acrid-bee'.\n└── UUID: ac68f58c-2104-4b56-aab2-2e02f61ef9be\n└── Parameters: {}\n└── Scheduled start time: 2023-03-13 10:47:14 PDT (in 2 weeks)\n└── URL: <no dashboard available>"

This appears to be surfacing b/c the library that's calculating the future time is not aware that Daylight Savings Time was made permanent last November... so PDT shouldn't be showing up for me 😬.

Checklist

  • This pull request references any related issue by including "closes <link to issue>"
    • If no issue exists and your change is not a small fix, please create an issue first.
  • This pull request includes tests or only affects documentation.
  • This pull request includes a label categorizing the change e.g. fix, feature, enhancement

@mjschock mjschock requested a review from cicdw as a code owner February 26, 2023 20:29
@netlify
Copy link

netlify bot commented Feb 26, 2023

Deploy Preview for prefect-docs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit e3db23f
🔍 Latest deploy log https://app.netlify.com/sites/prefect-docs/deploys/63fbc115a90a71000817d868
😎 Deploy Preview https://deploy-preview-8660--prefect-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Contributor

@zanieb zanieb left a comment

Choose a reason for hiding this comment

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

Thanks for contributing! Sounds like you should open an issue on the pendulum repository for the DST problem.

@zanieb zanieb merged commit 4939ae8 into PrefectHQ:main Feb 27, 2023
4 checks passed
@zanieb zanieb added the docs label Feb 27, 2023
ddelange added a commit to ddelange/prefect that referenced this pull request Mar 1, 2023
…aceful-agent

* 'patch-1' of https://github.com/ddelange/prefect: (25 commits)
  Enhancement: Track flow run id when generating task run results (PrefectHQ#8674)
  Fix loading of existing deployment descriptions from the server (PrefectHQ#8675)
  Update Dask logo (PrefectHQ#8669)
  Update `send_call` to `send_call_to_supervisor` (PrefectHQ#8653)
  Add Netlify Edge function to proxy to Segment (PrefectHQ#8657)
  Add security headers for docs (PrefectHQ#8655)
  Override analytics block instead of partial (PrefectHQ#8656)
  Add description flag to prefect deployment build CLI command (PrefectHQ#8603)
  Add support for yaml config strings to `KubernetesClusterConfig` (PrefectHQ#8643)
  Change lazy loads from joined to selectin (PrefectHQ#8659)
  Add timeout support to supervisors (PrefectHQ#8649)
  Minor markdown link fix in orchestration docs (PrefectHQ#8660)
  Allow more ports
  Bump @playwright/test from 1.30.0 to 1.31.1 in /ui (PrefectHQ#8663)
  Bump @prefecthq/prefect-ui-library from 1.1.9 to 1.1.10 in /ui (PrefectHQ#8661)
  Bump vite from 4.1.2 to 4.1.4 in /ui (PrefectHQ#8662)
  Bump eslint from 8.34.0 to 8.35.0 in /ui (PrefectHQ#8664)
  Fix of a typo - removed repeated word (PrefectHQ#8654)
  Enable DefaultAzureCredential authentication for Azure filesystem block (PrefectHQ#7513)
  Improve supervisor repr for debugging (PrefectHQ#8633)
  ...
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.

None yet

2 participants