GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — codegen should issue a diagnostic for a present but blank `name` on a field · Issue #795 · DFHack/df-structures

codegen should issue a diagnostic for a present but blank name on a field #795

New issue
Copy link
New issue
Copy link
Open
Open
codegen should issue a diagnostic for a present but blank name on a field#795
Copy link

Description

@ab9rf
ab9rf
opened on Aug 11, 2024
Issue body actions

alternatively, it should delete them and let the usual anon handling take place

right now, if an field element has name='' then codegen.out.xml will have a blank name attribute but no ld:anon-name attribute (as is generated if name is absent). while the headers get an anon name in the expected manner, codegen.out.xml fails to reflect what is generated in headers, which shouldn't happen

related discord discussion

Activity

  1. lethosor commented on Aug 11, 2024

    @lethosor
    lethosor
    on Aug 11, 2024
    Member
    More actions

    We should probably also disallow this in the schema, if that's possible.

    Reacted by Kelly Kinkade
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions
      You are visitor 01402520 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.