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

Remove deprecated signatures of pgr trsp #2699

Open
wants to merge 13 commits into
base: develop
Choose a base branch
from

Conversation

cvvergara
Copy link
Member

Fixes #2683 .

Changes proposed in this pull request:

  • Removal of
    • pgr_trsp(text,integer,double precision,integer,double precision,boolean,boolean,text)
    • pgr_trsp(text,integer,integer,boolean,boolean,text)
  • Removal of pgtap related tests
  • Removal of docqueries related pseudo tests
  • Update migration instructions
  • Fix upgrade test
  • Added a legacy.c for C deleted functions

@pgRouting/admins

@cvvergara cvvergara added TRSP Remove rm of deprecated function or internal function labels Dec 19, 2024
@cvvergara cvvergara added this to the Release 4.0.0 milestone Dec 19, 2024
@cvvergara cvvergara force-pushed the issue-2683-Remove-deprecated-signatures-of-pgr_trsp branch from bf0419f to 2e7ec6c Compare December 19, 2024 21:45
@cvvergara cvvergara force-pushed the issue-2683-Remove-deprecated-signatures-of-pgr_trsp branch from 2e7ec6c to 59973b5 Compare December 19, 2024 21:48
@cvvergara cvvergara requested a review from robe2 December 19, 2024 21:49
src/legacy/legacy.c Outdated Show resolved Hide resolved
Copy link
Member

@sanak sanak left a comment

Choose a reason for hiding this comment

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

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Remove rm of deprecated function or internal function TRSP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants