Skip to content

[WIP] Add a linter rule for nextlink that are not defined as urls #35395

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.

Sign up for GitHub

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

Jump to bottom
Draft
Copilot wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

[WIP] Add a linter rule for nextlink that are not defined as urls #35395

Copilot wants to merge 1 commit into from

Conversation

Copy link
Contributor

Copilot AI commented Jun 20, 2025

Thanks for assigning this issue to me. I'm starting to work on it and will keep this PR's description up to date as I form a plan and make progress.

Original issue description:

The guidelines are that nextLink should be an absolute URL. We should enforce this via linter rules so that we don't have these typed as strings.

Fixes #35375.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Jun 20, 2025 •
edited
Loading

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ This PR is labelled with ArcReview. For this PR to be merged, it must pass an ARC review and be labelled ArcSignedOff.
    Email the ARC board to request review per this Contact section.
  • ❌ The required check named Automated merging requirements met has failed. This is the final check that must pass. Refer to the check in the PR's 'Checks' tab for details on how to fix it and consult the aka.ms/ci-fix guide. In addition, refer to step 4 in the PR workflow diagram

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Jun 20, 2025

PR validation pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

Copilot AI requested a review from JoshLove-msft June 20, 2025 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JoshLove-msft JoshLove-msft Awaiting requested review from JoshLove-msft

At least 1 approving review is required to merge this pull request.

Assignees

@JoshLove-msft JoshLove-msft

Copilot code review Copilot

Labels

Alerts Management Analysis Services API Management ArcReview Authorization Automation Batch BatchAI Billing Cognitive - Language Cognitive Services Compute Container Instances Container Registry Container Service Fleet identifies PRs touching Azure Kubernetes Fleet Manager service spec Container Service Cosmos Customer Insights Data Factory Data Lake Analytics Data Lake Store Data Migration Device Update Device Update Service Devtestlab Event Grid Event Hubs HDInsight Import Export Insights Intune IoTHub KeyVault Load Test Service Azure Load Test Service Logic App Machine Learning Compute Machine Learning Maps Media Services Mobile Engagement Monitor - Exporter Monitor OpenTelemetry Exporter Monitor - Operational Insights Monitor Monitor, Monitor Ingestion, Monitor Query MySQL Network - CDN Network - DNS Network - Traffic Manager Network Notification Hub Operations Management Peering Policy Insights PostgreSQL Profile Azure API profile issues. Purview Recovery Services Backup Recovery Services Site-Recovery RecoveryServices Redis Cache Relay Scheduler Search Service Bus Service Fabric Service Map SQL Storage Storage Service (Queues, Blobs, Files) StorageSync Storsimple Stream Analytics Subscription Synapses TimeseriesInsights Web Apps

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add a linter rule for nextlink that are not defined as urls

2 participants

@JoshLove-msft