Skip to content

Remove deprecation of basic_zstring et al - #945

Merged
Jordan Maples (JordanMaples) merged 3 commits into
microsoft:masterfrom
JordanMaples:remove_deprecation
Oct 29, 2020
Merged

Jordan Maples (JordanMaples) merged 3 commits into
microsoft:masterfrom
JordanMaples:remove_deprecation

Conversation

@JordanMaples

Copy link
Copy Markdown
Contributor

basic_string_span and types derived from it were deprecated by CppCoreGuidelines 1680, not basic_zstring.

* azure-pipeline test

* nl @ eof

* trimming the pipeline and adding debug steps

* removing redundant lines

* change ctest to script cmd and remove debug

* removed bad char

* added dir change for ctest

* explicit output file and cmake standard

* test cat

* more ctest tests

* injecting failure in test for validation

* another test

* removing bad test

* massive matrix

* added parallel

* commenting everything but xcode out for testing purposes

* uncomment the other tests

* testing some variables

* rename

* changed macos versions

* adding one more layer of templates

* fixing jobs.yml

* idk what i'm doing

* slight modifications

* maybe some spaces will help

* removing 'variables.'

* another test

* adding back pr w/ autocancel

* adding failing test to validate error = failing task

* remove failing test

* trigger master

* nl in steps.yml
@JordanMaples
Jordan Maples (JordanMaples) merged commit e8978c0 into microsoft:master Oct 29, 2020
Dmitry Kobets (dmitrykobets-msft) added a commit that referenced this pull request Jun 28, 2023
With `string_span` having been deprecated (#931, #945) and removed (#1074), the header `<gsl/string_span>` now only contains the definitions for the `zstring` family. Update the name accordingly from `<gsl/string_span>` to `<gsl/zstring>`. The old header is now deprecated and should no longer be used and will be removed in some future release.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant