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.

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

feat: Render entry.go (first stage) #31

Merged
merged 36 commits into from
Mar 8, 2024
Merged

feat: Render entry.go (first stage) #31

merged 36 commits into from
Mar 8, 2024

Conversation

sebastianczech
Copy link
Contributor

Description

PR delivers new template to render entry.go with all required functions and changes in dependent parts of the code.

Motivation and Context

#30

How Has This Been Tested?

Automated tests are part of PR

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist

  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes if appropriate.
  • All new and existing tests passed.

@sebastianczech
new function to make indentation equal in list of strings
2155e70
@sebastianczech
execute new function creating definition of structs
d5ffed6
@sebastianczech
new function creating definition of structs (with tests)
dcb8f2e
@sebastianczech
fix logic after tests
7c5f19b
@sebastianczech
refactor code (remove duplication and extract func)
8171827
@sebastianczech
fix random order in keys for maps
7aad155
@sebastianczech
update testify package
b2bef13
@sebastianczech
new functions to generate functions for locations
cd267f2
@sebastianczech
use new functions for locations
e975329
@sebastianczech
extend template, do not concat strings to generate structs
b5095ac
@sebastianczech
extend template, do not concat strings to generate functions
a07ae67
@sebastianczech
clean code
2b7514c
@sebastianczech
tests for functions used while generating structs and funcs
5bc88c5
@sebastianczech
fix issues after tests
d9fdd83
@sebastianczech
remove not required spaces from templates
f1f2525
@sebastianczech
introduce new struct NameVariant for Location names and variables, ge…
b2fb714 
…t rid of camelCase function from template
@sebastianczech
Change Go SDK path for security policy rule
b535826
@sebastianczech
extend configuration file with settings for copying static assets
ea21716
@sebastianczech
static files from pango in develop branch
d2424f3
@sebastianczech
add function to copy static assets
0824d64
@sebastianczech
change go sdk path for service and address group
54f99e3
@sebastianczech
Merge branch 'render-location' into copy-static-files
f10d323
@sebastianczech
Add generic.Xml
f34380e
@sebastianczech
generate part of entry.go - struct Entry
73a3d43
@sebastianczech
add tests for names
a3b31d9
@sebastianczech
add generic package
5b07e8b
@sebastianczech
add name variants for params
2a8705c
@sebastianczech
start work on entry.go template
7b75458
@sebastianczech
add error handling for func AsEntryXpath
96e300a
@sebastianczech
Merge branch 'render-location' into copy-static-files
4b35c89
shinmog
shinmog approved these changes Mar 8, 2024
View reviewed changes
Copy link
Collaborator

@shinmog shinmog left a comment

Choose a reason for hiding this comment

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

lgtm, good starting point for entry rendering

@sebastianczech sebastianczech changed the title feat: Render entry.go feat: Render entry.go (first stage) Mar 8, 2024
@sebastianczech sebastianczech marked this pull request as ready for review March 8, 2024 08:41
Base automatically changed from copy-static-files to main March 8, 2024 09:59
@sebastianczech sebastianczech merged commit cdb02a7 into main Mar 8, 2024
1 check passed
@sebastianczech sebastianczech deleted the render-entry branch March 8, 2024 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pimielowski pimielowski pimielowski approved these changes

@shinmog shinmog shinmog approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sebastianczech @pimielowski @shinmog