Skip to content

Users/ahmadabdalla/hotfixes x#33

Merged
ahmadabdalla merged 16 commits intomainfrom
users/ahmadabdalla/hotfixes_x
Sep 2, 2023
Merged

Users/ahmadabdalla/hotfixes x#33
ahmadabdalla merged 16 commits intomainfrom
users/ahmadabdalla/hotfixes_x

Conversation

@ahmadabdalla
Copy link
Copy Markdown
Owner

@ahmadabdalla ahmadabdalla commented Sep 2, 2023

Description

Thank you for your contribution !

Please include a summary of the change and which issue is fixed.
Please also include the context.
List any dependencies that are required for this change.

Pipeline references

For module/pipeline changes, please create and attach the status badge of your successful run.

Pipeline

Type of Change

Please delete options that are not relevant.

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Update to documentation

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • My corresponding pipelines / checks run clean and green without any errors or warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (readme)
  • I did format my code

ahmadabdalla and others added 16 commits September 1, 2023 18:22
@ahmadabdalla
test
d5f5f6d
@ahmadabdalla
branch test
c027873
@ahmadabdalla
Updated SQL Server module to support dynamic location for Maintenance…
2cdeb1c 
… Configuration Property (Azure#3899)
@ahmadabdalla
push version change
3c877be
@ahmadabdalla
added updates
5bcae57
@ahmadabdalla
updated
1eb80cf
@AlexanderSehr
[CI Environment] Enabled the usage of both the old published module n…
40ab04c 
…ame, as well as the new PBR-aligned (Azure#3892)

* Simplified specs resource type identification

* Updated tempalte spec name handling

* Added a switch to control the name used for publishing via the settings.yml

* Temp switching setting to use classic name for testing

* Removed apparently redundant function

* Introduced temp change

* Introduced temp change

* Added conversion

* Added Microsoft to MS rename for template specs

* Disabled anything but publishing for testing

* Disabled anything but publishing for testing

* Disabled universal packaging as already tested

* Disabled api aligned

* Disabled most of gh pipeline for testing

* Refactored bool conversion to be more robust

* Added missing quote

* Update to latest

* Undid temp change

* Small fix

* Removed temp changes

* Added missing flag

* Added missing flag

* Re-enabled tests

* Readme fixes

* Readme fixes

* Readme fixes

* ReadMe Fixes
@AlexanderSehr
Fixed logic by introducing api name fetching
ac4927c
Push updated Readme file(s)
b5d0930
@ahmadabdalla
Merge remote-tracking branch 'upstream/main' into users/ahmadabdalla/…
6b9a627 
…hotfixes_x
@ahmadabdalla
try in moduleToPublish script
bd2b6a1
@ahmadabdalla
test
a0ef397
@ahmadabdalla
test
1747021
@ahmadabdalla
test
85ebf55
@ahmadabdalla
Merge remote-tracking branch 'upstream/users/ahmad/hotFixes' into use…
9b41983 
…rs/ahmadabdalla/hotfixes_x
@ahmadabdalla
test
faa6fd6
@ahmadabdalla ahmadabdalla merged commit 073875b into main Sep 2, 2023
@ahmadabdalla ahmadabdalla deleted the users/ahmadabdalla/hotfixes_x branch September 2, 2023 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@ahmadabdalla ahmadabdalla

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ahmadabdalla @AlexanderSehr